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 #51

Closed
wants to merge 1 commit into from

Conversation

ellipsis-dev[bot]
Copy link

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

Summary:

Issue: #49

Implementation:

  1. Split the 'get all PR data' endpoint
    • In the /src/pages/api/llm.ts file, split the 'get all PR data' endpoint into two separate endpoints: one for the diff data and one for all other data. This will involve creating two new API routes and modifying the existing one to remove the functionality that has been split out.
  2. Update the frontend to use the new endpoints
    • Update the frontend code in /src/pages/forms/fill/[id].tsx and /src/utils.ts to use the new endpoints. This will involve modifying the fetch calls to use the new endpoints and adjusting the data handling to accommodate the separate responses.

Report:

Step Result Details
1
Split the 'get all PR data' endpointSplit the 'get all PR data' endpoint into two separate endpoints: one for the diff data and one for all other data in the '/src/pages/api/llm.ts' file.
2
Update the frontend to use the new endpointsUpdated the frontend code in /src/pages/forms/fill/[id].tsx and /src/utils.ts to use the new endpoints.

Something look wrong?: If this Pull Request doesn't contain the expected changes, add more information to #49. 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 20, 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