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

feat: hide username from profile page #966

Merged

Conversation

attiyaIshaque
Copy link

@attiyaIshaque attiyaIshaque commented Feb 12, 2024

Description

Username visibility on the profile page is determined by the backend settings feature flag ENABLE_AUTO_GENERATED_USERNAME. If autoGeneratedUsernameEnabled is true then it will display the first name and last name initial of the user.

https://2u-internal.atlassian.net/browse/VAN-1834

Screenshots

Before After
Screenshot 2024-02-12 at 4 32 08 PM Screenshot 2024-02-22 at 1 38 31 PM

Copy link

codecov bot commented Feb 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (3fdd68b) 66.82% compared to head (28ea404) 67.09%.

Additional details and impacted files
@@                    Coverage Diff                    @@
##           simplify-registration     #966      +/-   ##
=========================================================
+ Coverage                  66.82%   67.09%   +0.27%     
=========================================================
  Files                         52       52              
  Lines                        850      857       +7     
  Branches                     176      178       +2     
=========================================================
+ Hits                         568      575       +7     
  Misses                       272      272              
  Partials                      10       10              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@attiyaIshaque attiyaIshaque force-pushed the attiya/VAN-1834-remove-username branch 2 times, most recently from 5a2fe2e to 9612bfa Compare February 13, 2024 07:12
@attiyaIshaque attiyaIshaque changed the title feat: remove username from profile page. feat: hide username from profile page on the bases of hide_username settings. Feb 13, 2024
@attiyaIshaque attiyaIshaque force-pushed the attiya/VAN-1834-remove-username branch 3 times, most recently from f0540a5 to e99d11f Compare February 20, 2024 10:08
@attiyaIshaque attiyaIshaque force-pushed the attiya/VAN-1834-remove-username branch from e99d11f to 3d6e6b9 Compare February 21, 2024 10:19
@attiyaIshaque attiyaIshaque force-pushed the attiya/VAN-1834-remove-username branch from 3d6e6b9 to 28ea404 Compare February 22, 2024 08:37
@zainab-amir zainab-amir changed the title feat: hide username from profile page on the bases of hide_username settings. feat: hide username from profile page Feb 23, 2024
@attiyaIshaque attiyaIshaque merged commit d06b6d7 into simplify-registration Feb 23, 2024
9 checks passed
@attiyaIshaque attiyaIshaque deleted the attiya/VAN-1834-remove-username branch February 23, 2024 07:55
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.

2 participants