Skip to content
This repository has been archived by the owner on Aug 1, 2024. It is now read-only.

feat: Use new edx-platform make migrate command. #1120

Merged
merged 1 commit into from
Jul 13, 2023

Conversation

dianakhuang
Copy link
Contributor

@dianakhuang dianakhuang commented Jul 12, 2023

Tries to make migrations run from devstack more consistent.

#1085


I've completed each of the following or determined they are not applicable:

  • Made a plan to communicate any major developer interface changes (or N/A)

@dianakhuang dianakhuang force-pushed the diana/remove-update-db branch from 3f3cc87 to 407412f Compare July 12, 2023 20:38
@dianakhuang dianakhuang changed the title Diana/remove update db feat: Use new edx-platform make migrate command. Jul 12, 2023
Copy link
Contributor

@robrap robrap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you confirm that you either tested the provisioning script, or copy/pasted and tested the individual docker-compose lines? Thanks.

Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Replaces some of the manual calls to `manage.py migrate`
with `make migrate` for more consistency with other IDAs.

#1085
@dianakhuang dianakhuang force-pushed the diana/remove-update-db branch from aaa3fc8 to c2ced80 Compare July 13, 2023 13:13
@dianakhuang
Copy link
Contributor Author

@robrap I have tested all of these on my local machine.

@dianakhuang dianakhuang merged commit c7de00f into master Jul 13, 2023
@dianakhuang dianakhuang deleted the diana/remove-update-db branch July 13, 2023 18:11
nsprenkle pushed a commit that referenced this pull request Nov 21, 2023
Replaces some of the manual calls to `manage.py migrate`
with `make migrate` for more consistency with other IDAs.

#1085
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.

2 participants