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

Dev to main Sync #932

Merged
merged 8 commits into from
Sep 19, 2024
Merged

Dev to main Sync #932

merged 8 commits into from
Sep 19, 2024

Conversation

shubhdevelop
Copy link
Contributor

@shubhdevelop shubhdevelop commented Sep 18, 2024

Date: 19th September, 2024

Developer Name: @shubhdevelop


Issue Ticket Number

PR's Going in These Sync

Description

  • add functionality for super_users to toggle their privileges

Documentation Updated?

  • Yes
  • No

Under Feature Flag

  • Yes
  • No

Database Changes

  • Yes
  • No

updated the data model documentation

Breaking Changes

  • Yes
  • No

Development Tested?

  • Yes
  • No

Screenshots

Screencast

Testing in staging

Screencast.from.19-09-24.02.03.40.AM.IST.webm
Screencast.from.19-09-24.02.05.00.AM.IST.webm

Under Feature flag

Screencast.from.19-09-24.03.39.14.PM.IST.webm
Screencast.from.19-09-24.03.44.52.PM.IST.webm

Test Coverage

Screenshot 1

image

image

Additional Notes

Link to the Design Doc

https://docs.google.com/document/d/17khsviClkNVNYH4Xy5Pkk98Xvrid1jH1jL9QtfGp7aw/edit?usp=sharing

shubhdevelop and others added 6 commits September 13, 2024 00:35
Changes
- added tests for it
- removes /debug pages from the dev feature flags
- Adds toast as on both success as well failure
- Adds test case covering the non-success reponse
from serves
[feat] Toggle Super_user privileges
@shubhdevelop shubhdevelop marked this pull request as ready for review September 19, 2024 00:16
@vinit717
Copy link
Member

Why you are removing FF as the backend changes are done now
Previously your feature is not tested as backend is not there

Copy link

cloudflare-workers-and-pages bot commented Sep 19, 2024

Deploying www-rds with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1bb3076
Status: ✅  Deploy successful!
Preview URL: https://470e0822.www-rds.pages.dev
Branch Preview URL: https://develop.www-rds.pages.dev

View logs

@shubhdevelop
Copy link
Contributor Author

Why you are removing FF as the backend changes are done now Previously your feature is not tested as backend is not there

Addressed it in the latest pr updated the PR description as well

@iamitprakash iamitprakash merged commit 62f2fa0 into main Sep 19, 2024
3 checks passed
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.

4 participants