Skip to content

issue #5501 fix: member country is not displayed in settings #5591

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

Conversation

gets0ul
Copy link
Collaborator

@gets0ul gets0ul commented Jun 26, 2021

#5501
use homeCountryCode or competitionCountryCode as displayed country in member settings page when country is not found in the basic_info traits

use homeCountryCode or competitionCountryCode as displayed country in member settings page when country is not found in the basic_info traits
Copy link
Collaborator

@luizrrodrigues luizrrodrigues left a comment

Choose a reason for hiding this comment

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

@gets0ul If try to edit any field will show error message in Country field:
https://i.gyazo.com/e9c82a736649ac24d292a15cb5fbe0f6.mp4

@sushilshinde
Copy link
Collaborator

@gets0ul If try to edit any field will show error message in Country field:
https://i.gyazo.com/e9c82a736649ac24d292a15cb5fbe0f6.mp4

@gets0ul can you finish this today?

@gets0ul
Copy link
Collaborator Author

gets0ul commented Jun 29, 2021

@sushilshinde Yes, will update it ASAP

update state when list of all countries for mapping has been received from back end.
@gets0ul
Copy link
Collaborator Author

gets0ul commented Jun 29, 2021

@luizrrodrigues @sushilshinde Updated

@luizrrodrigues luizrrodrigues self-requested a review June 29, 2021 07:16
@luizrrodrigues
Copy link
Collaborator

@gets0ul Thanks, checking.

@luizrrodrigues luizrrodrigues merged commit 36fce9e into topcoder-platform:fix-country-setting-page Jun 29, 2021
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.

3 participants