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

Forum Commands: Subscriptions #7254

Closed
rbennettcw opened this issue Mar 26, 2024 · 2 comments
Closed

Forum Commands: Subscriptions #7254

rbennettcw opened this issue Mar 26, 2024 · 2 comments
Assignees

Comments

@rbennettcw
Copy link
Contributor

rbennettcw commented Mar 26, 2024

Child of #7203

Acceptance Criteria

  • There should be commands for the following actions
    • Subscriptions
      • CreateSubscription
      • DeleteSubscription
      • UpdateSubscription (enable/disable)
  • There should be new tRPC and Express handlers
  • The UI should be updated to use the new routes
  • There should be async handling (notifications, analytics)
  • There should be lifecycle tests (minimum 90% coverage)
  • BDD Cases
    • TODO
@rbennettcw rbennettcw self-assigned this Mar 26, 2024
@rbennettcw
Copy link
Contributor Author

@timolegros This is the ticket for moving the subscription-related logic into the framework as commands. Please confirm if these routes are still relevant considering the recent changes to the notifications system.

@timolegros
Copy link
Collaborator

@timolegros This is the ticket for moving the subscription-related logic into the framework as commands. Please confirm if these routes are still relevant considering the recent changes to the notifications system.

All of these routes will probably be deprecated. We can probably close this ticket in favor of #6946.

@rbennettcw rbennettcw closed this as not planned Won't fix, can't repro, duplicate, stale Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants