-
Notifications
You must be signed in to change notification settings - Fork 5
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
Clean out Amplify's legacies #33
Comments
@rishabkumar7 mentioned that some new changes (the ones seen above) should be pushed to the current app. Reconnecting the Creating a new repo with a 1:1 copy of this' |
The old gatsby site is moved to 100daysOfCloudLegacy repo. |
The new repository that's connected to the Amplify App is 100DaysOfCloudLegacy - it holds the original (old) Amplify code but it got seperated from this repository. This repository is murky on branches and was therefore split up into 100DaysOfCloudLegacy & 100DaysOfCloudBackend. This repository was also renamed to Keep issue open until the new 100DaysOfCloudBackend repository holds a copy of the latest backend SAM app (template). |
Agree with this, it should be also linked to #34. |
The Amplify app has been completely disconnected from this repository because of the repo name change. It did not follow the changes in this repository and no matter what happens here (either on
main
orstaging
), it will not be reflected on the current Amplify app. As seen below, the latest update to the live app was f6be3f9What I don't understand at all is why the last build (Build 4) is saying it happened yesterday (today is 29th) when that merge was 11 days ago.
I just disabled auto build for the
main
environment as well for good measure, since it did not break or change anything for thestaging
environment when that got disabled a while back.Conclusion
main
branch of this repository or Amplify code that's in there, it is isolated from everything. We should clear everything out in themain
branch to avoid confusion and put a well-detailed README of where things are at.dev
asstaging
is not what we're doing right now and it is just confusing. Themain
branch would hold the explanation README and thedev
branch would hold all our backend code for now. When production will be deployed, themain
branch will be populated and a CI/CD pipeline will handle everything from the get-go.dev
branch is an exact copy, deletestaging
branch.The text was updated successfully, but these errors were encountered: