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: backend projects not submitting #38900

Conversation

moT01
Copy link
Member

@moT01 moT01 commented May 26, 2020

Should fix number 3 in this comment

Some issues with the form values not being updated correctly. Making this a draft for now, some of the things I deleted may be needed yet. Want to make sure.

The change that made things work is on like 181 of show.js

  • I have read freeCodeCamp's contribution guidelines.
  • My pull request has a descriptive title (not a vague title like Update index.md)
  • My pull request targets the master branch of freeCodeCamp.
  • All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

@gitpod-io
Copy link

gitpod-io bot commented May 26, 2020

Copy link
Contributor

@ojeytonwilliams ojeytonwilliams left a comment

Choose a reason for hiding this comment

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

I finished the clean up (there was still some backendFormValue code), but it all seems to be working nicely.

All the form submission goes through SolutionForm so there's no need to track backend challenge, backend project or frontend project form values separately any more. At least that's my understanding.

@moT01 moT01 marked this pull request as ready for review May 26, 2020 12:37
@moT01 moT01 requested a review from a team May 26, 2020 12:37
@raisedadead raisedadead added platform: learn UI side of the client application that needs familiarity with React, Gatsby etc. release: next/beta labels May 26, 2020
@raisedadead raisedadead merged commit a001041 into freeCodeCamp:next-python-projects May 26, 2020
@moT01 moT01 deleted the fix/something-on-python-branch branch May 26, 2020 13:54
raisedadead pushed a commit that referenced this pull request May 26, 2020
Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
raisedadead pushed a commit that referenced this pull request May 27, 2020
Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
raisedadead pushed a commit that referenced this pull request May 27, 2020
Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
abbathaw pushed a commit to abbathaw/freeCodeCamp that referenced this pull request Jul 24, 2020
Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform: learn UI side of the client application that needs familiarity with React, Gatsby etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants