Closed tpwrules closed 4 months ago
@tpwrules, thanks for sharing this! We very much appreciate your PR and would happy to receive future contributions from you! Indeed, we have not kept pace with documenting our new setup approaches that we started refactoring in late 2021.
I just assigned @ptomasula to review your PR.
@ptomasula, as a followup to our call today, I decided to redirect this PR to the new tpw_fix_setup
feature branch that I just created from develop
. This gives you the ability to resolve the merge conflicts, complete the merge, then test it using CDK, without interfering with your current branches.
We'll consider this step 1 of 2(ish) in addressing the larger issue (and opportunity) raised by @tpwrules with:
I could not find any guidelines on contributing back to this project, so I hope this is received in good faith.
This fixes some issues I found trying to initially set up an instance of this application. Please see the commit messages for details.
There is still a lot of work to be done with documentation and initial database setup but now that I have the site loading I can work on that.