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

Fix(accounts): show challenge info on user profile #10041

Merged
merged 1 commit into from
Aug 2, 2016
Merged

Fix(accounts): show challenge info on user profile #10041

merged 1 commit into from
Aug 2, 2016

Conversation

BerkeleyTrue
Copy link
Contributor

closes #9954

This changes the behavior of the user profile page to pull the current
challenge info from our challenge map and overwrite the user challenge.
This should also make name changes point to the correct challenge
regardless of the info saved to the user profile
@BerkeleyTrue BerkeleyTrue added the status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending. label Aug 2, 2016
@raisedadead
Copy link
Member

raisedadead commented Aug 2, 2016

Just to confirm this will currently fix the empty table of challenges and not actually show the solution right?

image

Should I raise a separate issue for that? Or do we have something already?
Apart from that, I think this is okay to merge.

@BerkeleyTrue
Copy link
Contributor Author

Yes. If there isn't already an issue please open one. Thanks @raisedadead !

@raisedadead raisedadead merged commit 0a90d0f into freeCodeCamp:staging Aug 2, 2016
@raisedadead raisedadead removed the status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending. label Aug 2, 2016
@BerkeleyTrue BerkeleyTrue deleted the regression/add-current-challenge branch September 28, 2016 23:24
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