-
Notifications
You must be signed in to change notification settings - Fork 1
Json templates #29
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
Json templates #29
Conversation
…outer Implement basic routing and protected routes stub
… into StevenYan/implement-user-authentication
Homepage created
Refactored Pages
Manual Entry Page
Integrated the frontend & backend for logbook page and log history page.
fix: added title to adult cardiac example
Sarah wang/upload photo changes
…books-button Update ShopBooksCard button to link to flowleaflets store
…potentially be used in app.py
TonyLiu0226
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work! Just a couple of minor suggestions for next time in the comments. Feel free to merge in, then once you do that change the port back to 5000 from 8000.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
try to avoid adding these in the future, maybe include in gitignore
|
|
||
| if __name__ == "__main__": | ||
| app.run(host='0.0.0.0',port=5000) No newline at end of file | ||
| app.run(host='0.0.0.0',port=8000) No newline at end of file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please keep this at 5000
Updated Json object for logbook templates. Changed empty strings to null values and added line items. Still working on the testing functions.