-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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: add work experience field. #34034
Conversation
@@ -268,7 +268,8 @@ def _get_extended_profile_fields(): | |||
platform_name=configuration_helpers.get_value("PLATFORM_NAME", settings.PLATFORM_NAME) | |||
), | |||
"profession": _("Profession"), | |||
"specialty": _("Specialty") | |||
"specialty": _("Specialty"), | |||
"work_experience": _("Work experience") |
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.
This should be covered in tests. test_setting_views.py
in test_context
test
ea58b88
to
4ff46ba
Compare
4ff46ba
to
d7341bd
Compare
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
2U Release Notice: This PR has been deployed to the edX production environment. |
1 similar comment
2U Release Notice: This PR has been deployed to the edX production environment. |
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
2U Release Notice: This PR has been deployed to the edX production environment. |
Description
I have added a work experience field that will be shown on the front-end authn welcome page and also on the legacy account setting page.
Ticket
https://2u-internal.atlassian.net/browse/VAN-1788
How Has This Been Tested?
you can add settings in django admin site configuration like this
and the filed will be shown in welcome page of auth mfe and account settings.