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

Use new GET /subscription_user endpoint to populate subscription state #922

Open
4 tasks
eleanorreem opened this issue May 23, 2024 · 0 comments
Open
4 tasks
Labels
complexity: advanced Time needed to do this ticket will be large e.g. 2-3+ days help wanted Extra attention is needed maintenance Maintenance / chore work priority: high Should be prioritized immediately, likely blocking other tasks. priority: soon Should be prioritized soon. state: approved Ready to go. Not blocked or pending.

Comments

@eleanorreem
Copy link
Contributor

eleanorreem commented May 23, 2024

Overview

We want to start breaking up the GET /user/me request and only fetch information on the pages it is needed. This is starting with subscriptions. We are getting subscription information on every page when we actually only need it on the /subscription/whatsapp page. Please note this is potentially a large refactor.

Note this ticket is related to this issue on the backend.

Action Items

  • Take a look at the app/userSlice.ts
  • Stop populating subscription data on the userSlice from the /user/me request. Instead, populate subscriptions state only on the /subscription/whatsapp page using the new endpoint GET /subscription_user.
  • Ensure you can successfully subscribe and unsubscribe
  • Ensure Cypress tests pass
@eleanorreem eleanorreem added complexity: advanced Time needed to do this ticket will be large e.g. 2-3+ days maintenance Maintenance / chore work state: blocked Task is blocked. labels May 23, 2024
@kyleecodes kyleecodes added this to the 02. Roadmaps milestone May 27, 2024
@eleanorreem eleanorreem added state: approved Ready to go. Not blocked or pending. priority: soon Should be prioritized soon. and removed state: blocked Task is blocked. labels Sep 21, 2024
@kyleecodes kyleecodes added help wanted Extra attention is needed hacktoberfest Hacktoberfest issues labels Oct 4, 2024
@kyleecodes kyleecodes removed the hacktoberfest Hacktoberfest issues label Nov 6, 2024
@eleanorreem eleanorreem added the priority: high Should be prioritized immediately, likely blocking other tasks. label Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity: advanced Time needed to do this ticket will be large e.g. 2-3+ days help wanted Extra attention is needed maintenance Maintenance / chore work priority: high Should be prioritized immediately, likely blocking other tasks. priority: soon Should be prioritized soon. state: approved Ready to go. Not blocked or pending.
Projects
Status: To-Do Bloom Frontend
Development

No branches or pull requests

2 participants