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

[ui] add ui update form route #119

Closed
signebedi opened this issue Mar 29, 2024 · 1 comment
Closed

[ui] add ui update form route #119

signebedi opened this issue Mar 29, 2024 · 1 comment

Comments

@signebedi
Copy link
Owner

Child of #30, #116, and #42. Pull the user document and pass it as an HTML-friendly dict to the Jinja template. Unset values should be nulled in the HTML - so no default values should be used. We can pass the whole document to the API route and document database; only the changes will be captured, notwithstanding some funky data mutation stemming from type conversion.

@signebedi
Copy link
Owner Author

It's a legitimate question: do we plan to pass the existing data to the html generator function, or is the javascript going to do that? I think the javascript is going to need to do that, and route everything through the API routes (eg. pull the data and overwrite the default values). Otherwise, we are breaking core design assumptions that the UI will be built atop a fully Restful application...

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

No branches or pull requests

1 participant