-
Notifications
You must be signed in to change notification settings - Fork 25
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
Allow update profile functionality from settings page #22
Comments
That would be awesome and probably after this no phone needed anymore. From npm tinder:
That is working right? |
Hm, that's actually really nice haha ! My only concern is that when we move to widescreen those icons will be spaced out too far. Maybe we stick them all beside eachother on the right hand side, with a logout icon as well, and then some branding in the left corner |
And yes, .updatePreferences is working, @stuwil is sniffing out a lot of the endpoints right now so there will be more added that will be used in this section soon |
Most of this is in #67, adding to release milestone. |
I added edit bio functionality to the already existing profile page with #157. Planning to add photo uploading when I have the time. (sorry if this is the wrong issue to discuss this in? It was linked to from Development Priorities with a link titled "Add edit bio functionality") |
We either need to create a new page for updating a profile, or drop it on the settings page
This page should allow changing things like:
A lot of these endpoints will either need to be sniffed, or found in unofficial documentation elsewhere on the web and then added to the Tinder module, we should be able to use what I believe is the
update
API call to update the data, we just need to know how to pass the data.The text was updated successfully, but these errors were encountered: