afids / afids-validator

Validator for the anatomical fiducial placement protocol
https://validator.afids.io
GNU General Public License v3.0
2 stars 4 forks source link

MAINT: Refactor into project with a blueprint and application factory #145

Closed tkkuehn closed 3 years ago

tkkuehn commented 3 years ago

Proposed changes

This PR changes the repo into a project structure, refactors the views into a blueprint, and generates the app using an application factory, all as proposed in the Flask docs for a larger project. I think I changed the procfile correctly, but we'll have to keep an eye on Heroku once this gets deployed.

Types of changes

What types of changes does your code introduce? Put an x in the boxes that apply

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you are unsure about any of the choices, don't hesitate to ask!

Notes

All PRs will undergo the unit testing before being reviewed. You may be requested to explain or make additional changes before the PR is accepted.

_PR template was adopted from appium_

tkkuehn commented 3 years ago

This looks good to me now! Should we link this with #140?

Yeah, I think so!

kaitj commented 3 years ago

Great, linked with 140. I'll wait to merge until next meeting in case deployment breaks with all the changes.