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

🗑️ Removed unused functions and relevant code #85

Merged
merged 6 commits into from
Oct 29, 2024

Conversation

babakamyljanovssw
Copy link
Member

Relates: SSWConsulting/SSW.Rules#1567

This pull request involves significant refactoring and removal of reaction-related functionality from the codebase. The most important changes include the deletion of the Reaction class and associated functions, as well as updates to the RulesDbContext and Program files to reflect these removals.

Removal of Reaction-related functionality:

  • Entire Reaction class and ReactionType enum have been removed.
  • Deleted the GetAllReactionsFunction class and its implementation.
  • Deleted the GetReactionsFunction class and its implementation.
  • Deleted the ReactFunction class and its implementation.
  • Deleted the RemoveReactionFunction class and its implementation.

Updates to RulesDbContext and Program:

  • Removed the Reactions property from the RulesDbContext class.
  • Removed the Reaction container configuration and its import statement.

Documentation updates:

  • Removed all endpoints related to reaction functions and user comments account functions.

Copy link
Member

@bradystroud bradystroud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@babakamyljanovssw babakamyljanovssw merged commit 9eee776 into main Oct 29, 2024
3 checks passed
@babakamyljanovssw babakamyljanovssw deleted the remove-functions branch October 29, 2024 02:17
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.

2 participants