Skip to content
This repository has been archived by the owner on Sep 21, 2024. It is now read-only.

[2] Prevent additional state properties on Profile page from being POSTed #169

Open
MitchellDill opened this issue Aug 16, 2019 · 0 comments
Labels
bug Something isn't working
Milestone

Comments

@MitchellDill
Copy link
Contributor

MitchellDill commented Aug 16, 2019

Currently, the updateAccountData is being invoked on the profile page with the profile page's entire state object, which includes some page-specific stuff we don't wanna send to the database (passwordIsHidden, etc). We'll just want to make sure we only send the user object to the database!

This will likely intersect somewhat with #165 --Brandon is working to reorganize the POST/setState flow at app level, so that'll probably be relevant.

Bugfix Outline

  1. Refactor profile page info updates to ensure only the updated user object to the app level for state update/POSTing
  2. Check DB to ensure no additional fields have been added to the user's document
  3. Ensure that when returning the profile page, all the fields you've udpdated are still updated
@MitchellDill MitchellDill added the bug Something isn't working label Aug 16, 2019
@MitchellDill MitchellDill added this to the Profile Page milestone Aug 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant