Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Ellipsis] Split GitHub router's admin endpoint into two #58

Closed
wants to merge 1 commit into from

Conversation

ellipsis-dev[bot]
Copy link

@ellipsis-dev ellipsis-dev bot commented Jan 21, 2024

Summary:

Issue: #55

Implementation:

  1. Create new endpoints
    • In the /src/pages/api directory, create two new files: diffData.ts and otherData.ts. These files will contain the code for the new endpoints. The diffData.ts endpoint will handle requests for diff data, and the otherData.ts endpoint will handle requests for all other data.
  2. Implement diffData endpoint
    • In the diffData.ts file, implement the endpoint to fetch the diff data. This endpoint should accept a PR id as a parameter and return the diff data for that PR. Use the existing llm.ts file as a reference for how to structure the endpoint.
  3. Implement otherData endpoint
    • In the otherData.ts file, implement the endpoint to fetch all other data. This endpoint should accept a PR id as a parameter and return all other data for that PR. Use the existing llm.ts file as a reference for how to structure the endpoint.
  4. Update fetch calls in frontend
    • Update the fetch calls in the frontend to use the new endpoints. The fetch calls are located in the callLLM function in the utils.ts file and in the handleSubmit function in the fill/[id].tsx file. Replace the /api/llm endpoint with the appropriate new endpoint (/api/diffData or /api/otherData).

Report:

Step Result Details
1
Create new endpointsCreated new endpoints diffData.ts and otherData.ts in /src/pages/api directory.
2
Implement diffData endpointImplemented the diffData endpoint in the diffData.ts file.
3
Implement otherData endpointImplemented the otherData endpoint in the otherData.ts file.
4
Update fetch calls in frontendUpdated the fetch calls in the frontend to use the new endpoints.

Something look wrong?: If this Pull Request doesn't contain the expected changes, add more information to #55. Then, add the ellipsis:create label to try again. For more information, check the documentation.

Generated with ❤️ by ellipsis.dev

@nsbradford nsbradford closed this Jan 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant