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.

Sign up for GitHub

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

Jump to bottom

feat(Component): Add new endpoint that allows user to subscribe and unsubscribe to a component #2737

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feat(Component): Add new endpoint that allows user to subscribe and unsubscribe to a component #2737

wants to merge 1 commit into from
+43 −1

Conversation

hoangnt2
Copy link
Contributor

Add new endpoint that allows user to subscribe and unsubscribe to a component

  • If the user has previously subscribed to that component, they will unsubscribe from that component
  • If the user has not previously subscribed to that component, they will subscribe from that component

How To Test?

Checklist

Must:

  • All related issues are referenced in commit messages and in PR

@hoangnt2 hoangnt2 added needs code review needs general test This is general testing, meaning that there is no org specific issue to check for REST New-UI Level for the API and UI level changes for the new-ui labels Nov 22, 2024
@hoangnt2 hoangnt2 mentioned this pull request Nov 22, 2024
Merged
smrutis1
smrutis1 approved these changes Nov 25, 2024
View reviewed changes
Copy link
Contributor

@smrutis1 smrutis1 left a comment

Choose a reason for hiding this comment

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

Code looks good.

@hoangnt2 hoangnt2 force-pushed the release/feat_component-subscription branch from 505ede8 to 794b87f Compare November 26, 2024 02:52
@GMishx GMishx self-assigned this Nov 27, 2024
@GMishx GMishx added this to the Release - 20.0.0 milestone Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@smrutis1 smrutis1 smrutis1 approved these changes

@heliocastro heliocastro Awaiting requested review from heliocastro heliocastro is a code owner

@GMishx GMishx Awaiting requested review from GMishx GMishx is a code owner

@KoukiHama KoukiHama Awaiting requested review from KoukiHama KoukiHama is a code owner

@arunazhakesan arunazhakesan Awaiting requested review from arunazhakesan arunazhakesan is a code owner

@ag4ums ag4ums Awaiting requested review from ag4ums ag4ums is a code owner

Assignees

@GMishx GMishx

Labels
needs general test This is general testing, meaning that there is no org specific issue to check for New-UI Level for the API and UI level changes for the new-ui REST
Projects
None yet
Milestone
Release - 20.0.0
Development

Successfully merging this pull request may close these issues.
4 participants
@hoangnt2 @smrutis1 @GMishx @duonglq-tsdv