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

Simplify contentprocessor service #2

Merged
merged 2 commits into from
Jul 30, 2024
Merged

Conversation

wmeints
Copy link
Owner

@wmeints wmeints commented Jul 30, 2024

Closes #1

Simplify the contentprocessor service by removing command handlers and wiring API endpoints directly to functions.

  • Remove GenerateDescriptionCommandHandler.cs, GeneratePaperScoreCommandHandler.cs, SummarizePageCommandHandler.cs, and SummarizePaperCommandHandler.cs files.
  • Modify Program.cs to remove dependency injection for command handlers and update API endpoints to directly call the corresponding functions.
  • Ensure dependency injection for functions remains in Program.cs.

For more details, open the Copilot Workspace session.

wmeints added 2 commits July 30, 2024 17:44
Closes #1

Simplify the contentprocessor service by removing command handlers and wiring API endpoints directly to functions.

* Remove `GenerateDescriptionCommandHandler.cs`, `GeneratePaperScoreCommandHandler.cs`, `SummarizePageCommandHandler.cs`, and `SummarizePaperCommandHandler.cs` files.
* Modify `Program.cs` to remove dependency injection for command handlers and update API endpoints to directly call the corresponding functions.
* Ensure dependency injection for functions remains in `Program.cs`.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/wmeints/paperboy/issues/1?shareId=XXXX-XXXX-XXXX-XXXX).
@wmeints wmeints merged commit 3fbb43c into main Jul 30, 2024
1 check passed
@wmeints wmeints deleted the simplify-contentprocessor branch July 30, 2024 17:19
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.

Simplify contentprocessor service
1 participant