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

docs: add heroku sql import instructions + db cleanup script #3103

Merged
merged 8 commits into from
Dec 16, 2019

Conversation

TomGranot
Copy link

@TomGranot TomGranot commented Oct 19, 2019

A few notes, as to why this is WIP:

  1. When actually trying to log into my docker heroku app I see: Trying to get property 'latest_version' of non-object. Note that I attempted to import data from an old version of Monica to a new one.
  2. Which begs the question: what happens if I export from an older version of Monica and import to a new one? The SQL breaks and gives errors like in the bottom of my PR. This is supposedly addressable by migrations, but I'm not sure what's the best way to attack it. It's also possible to just instruct users to first update to the latest version of Monica and then do the export, which I did in this doc.
  3. Wasn't aware of the commit guidelines, and thus didn't follow the commit guidelines, sorry about that!

BTW, I still have the dynos up if any one of you is up for some investigation.

@TomGranot TomGranot changed the title Add Heroku SQL import instructions + script [WIP] Add Heroku SQL import instructions + script Oct 19, 2019
@TomGranot TomGranot changed the title [WIP] Add Heroku SQL import instructions + script [WIP] Add Heroku SQL import instructions + DB truncation script Oct 19, 2019
@TomGranot
Copy link
Author

@asbiin btw, this is how I got to the issue of the reninders, while trying to fix this up.

@TomGranot TomGranot changed the title [WIP] Add Heroku SQL import instructions + DB truncation script docs: add heroku sql import instructions + db cleanup script Oct 23, 2019
@asbiin asbiin merged commit 44fa487 into monicahq:master Dec 16, 2019
@asbiin
Copy link
Member

asbiin commented Dec 16, 2019

thank you @Tomgs ! It ihas taken a long time, but this is now merged.

@github-actions
Copy link

This pull request has been automatically locked since there
has not been any recent activity after it was closed.
Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants