-
Notifications
You must be signed in to change notification settings - Fork 132
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
[My Site Migration] migration for user status feature (Part - 1) #975
base: develop
Are you sure you want to change the base?
[My Site Migration] migration for user status feature (Part - 1) #975
Conversation
add the screenshot of the test coverage you've written, not just passing of all the test cases |
add screen recording showing mobile responsiveness |
added the screenshots in Part-2 which contains all the tests for the feature |
added the screen recording! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please revert the commented change and other than that everything lgtm
Please update the branch. |
Date: 09-01-25
Developer Name: @MayankBansal12
Issue Ticket Number:-
Description:
This is the Part 1 for the user status migration PR. Refer to PR #963 for the complete version.
In this part, routes, controllers, components for user-status and user-status-modal and related files have been migrated.
Is Under Feature Flag
Database changes
Breaking changes (If your feature is breaking/missing something please mention pending tickets)
Is Development Tested?
Tested in staging?
Add relevant Screenshot below ( e.g test coverage etc. )
mobile-responsiveness.mp4
working-demo.mp4