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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

馃摎 Documentation: Clarify migrations are not always required #3624

Closed
2 tasks done
stnguyen90 opened this issue Jul 28, 2022 · 10 comments
Closed
2 tasks done

馃摎 Documentation: Clarify migrations are not always required #3624

stnguyen90 opened this issue Jul 28, 2022 · 10 comments
Assignees
@us3r64bit
Labels
good first issue Good for newcomers
Milestone
1.0.0

Comments

@stnguyen90
Copy link
Contributor

馃挱 Description

Even though the upgrade docs state:

You do not need to run migration when upgrading to a minor version. For example, going from 0.13.0 to 0.13.2 would not require running migration.

When following along with the instructions, the "Running the Migration" just says:

We can now start the migration. Navigate to the appwrite directory where your docker-compose.yml is present and run the following command

It would be clearer to emphasize, again, to run the migration only if required.

Also, the link on the "Running the Migration" header doesn't work.

Quote from discord:

I get confused, since it does say not to do it, then below it lays it out as the next step.

馃憖 Have you spent some time to check if this issue has been raised before?

  • I checked and didn't find similar issue

馃彚 Have you read the Code of Conduct?
@stnguyen90 stnguyen90 added the good first issue Good for newcomers label Jul 28, 2022
@stnguyen90 stnguyen90 added this to the 0.15.4 milestone Jul 28, 2022
@us3r64bit
Copy link

Please Assign me this issue

@stnguyen90
Copy link
Contributor Author

@arth2002, assigned! Thanks for your help! 馃檹

@us3r64bit
Copy link

@stnguyen90 Sir,
sorry to bother you
but I can't find the file which needs to be changed
I find the "running the migration" page which is https://appwrite.io/docs/upgrade
but I couldn't find it in the docs folder of the project.

Can you please help me with this?

@stnguyen90
Copy link
Contributor Author

@arth2002, here you go! https://github.com/appwrite/docs/blob/main/app/views/docs/upgrade.phtml

@us3r64bit
Copy link

Sir, I'm a little confused
so do I need to create a pull request on the appwrite/docs repository for this issue?

@2002Bishwajeet
Copy link
Contributor

Hey @arth2002 ,
Yes, you need to create a PR on the appwrite/docs repo. You can refer to this issue in the PR

@us3r64bit
Copy link

@stnguyen90 sir
I've made PR
Please check here--> appwrite/docs#230
Let me know if there is some change to it
Thank You 馃槃

@us3r64bit
Copy link

@stnguyen90 Sir, You can close this issue
It has been solved here
Thanks for your help 馃槃

@stnguyen90 stnguyen90 mentioned this issue Aug 16, 2022
Merged
@stnguyen90
Copy link
Contributor Author

@arth2002, thanks for your contribution! 馃檹

@stnguyen90 stnguyen90 modified the milestones: 0.15.4, 1.0.0 Sep 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@us3r64bit us3r64bit

Labels
good first issue Good for newcomers
Projects
None yet
Milestone
1.0.0
Development

No branches or pull requests
4 participants
@stnguyen90 @2002Bishwajeet @us3r64bit and others