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 first and last name fields for user #311

Merged
merged 3 commits into from
Oct 17, 2023

Conversation

ahmed-arb
Copy link

Description

This changes signup form to take first and last name instead of full name. The signup form will look like this:
Screenshot 2023-10-11 at 1 47 01 PM

Instead of

Screenshot 2023-10-11 at 1 46 17 PM

And if users want to change their name afterwards accounts settings page will be changed to take first and last name separately as well:

Screenshot 2023-10-11 at 1 53 34 PM

Additionally, sysadmin report, Download list of all users, will be changed to have first and last name.

@ahmed-arb ahmed-arb requested a review from ziafazal October 11, 2023 08:57
Copy link

@ziafazal ziafazal left a comment

Choose a reason for hiding this comment

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

👍

@ziafazal
Copy link

@ahmed-arb looks good please squash commits while merging.

@ahmed-arb ahmed-arb merged commit 6b17bff into develop Oct 17, 2023
@ahmed-arb ahmed-arb deleted the ahmed-arb/add-name-fields-to-user branch October 17, 2023 06:54
ahmed-arb added a commit that referenced this pull request Oct 25, 2023
* feat: Add GA 4 support to edX platform

feat: add a snippet to public_video_share_embed.html

feat: add google_analytics_4_id template tag for main_django.html

fix: add js_escaped_string filter for ga_4_id

* removed extra file public_video_share_embed.html

* Use first and last name fields for user (#311)

* feat: ask first and last name on account creation

* feat: add first and last name in sysadmin report

* feat: add first and last name in accounts page

---------

Co-authored-by: Sagirov Eugeniy <sagirov19@gmail.com>
Co-authored-by: Zain Virk <zain.virk@edly.io>
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