Skip to content

Commit 3fa700b

Browse files
committed
temp: it's Ulmo now, not Teak
1 parent a61585f commit 3fa700b

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

docs/decisions/0020-authoring-as-one-app.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ In one pull request, we would:
3939
4. Model Migration Plan
4040
~~~~~~~~~~~~~~~~~~~~~~~
4141

42-
Migrating models across apps is tricky. This plan assumes that people will either have a new install or run migrations from Teak or the current "main" branch, both of which have the same models/schema at the time of this writing (v0.30.2).
42+
Migrating models across apps is tricky. This plan assumes that people will either have a new install or run migrations from Ulmo or the current "main" branch, both of which have the same models/schema at the time of this writing (v0.30.2).
4343

4444
The new ``authoring`` app's initial migration will detect whether it is a new install or an update to an existing one and either create the tables or simply repoint the models to the existing schema. The next migration will rename the tables to have a common ``oel_authoring`` prefix.
4545

openedx_learning/apps/authoring/migrations/0001_initial.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
This migration has two modes it needs to run in:
33
44
1. Existing installs that have migration data that is current through 0.30.2
5-
(bundled with the Teak release).
5+
(bundled with the Ulmo release).
66
2. New installs.
77
"""
88
import uuid
@@ -22,7 +22,7 @@ class BootstrapMigrations(SeparateDatabaseAndState):
2222
def __init__(self, operations):
2323
return super().__init__(database_operations=operations, state_operations=operations)
2424

25-
def has_teak_release_tables(self):
25+
def has_ulmo_release_tables(self):
2626
"""
2727
There are three possible outcomes:
2828
@@ -31,7 +31,7 @@ def has_teak_release_tables(self):
3131
2. The database has no migrations of those earlier apps: Return False.
3232
3. The database has *some* but not all of the migrations we expect:
3333
Raise an error. This can happen if someone tries to upgrade and skips
34-
the Teak release, e.g. Sumac -> Verawood directly.
34+
the Ulmo release, e.g. Teak -> Verawood directly.
3535
"""
3636
expected_migrations = {
3737
"oel_collections": "0005_alter_collection_options_alter_collection_enabled",
@@ -50,20 +50,20 @@ def has_teak_release_tables(self):
5050

5151
if MigrationRecorder.Migration.objects.filter(app="oel_publishing").exists():
5252
raise RuntimeError(
53-
"Migration could not be run because database is in a pre-Teak "
54-
"state. Please upgrade to Teak (openedx_learning==0.30.2) "
53+
"Migration could not be run because database is in a pre-Ulmo "
54+
"state. Please upgrade to Ulmo (openedx_learning==0.30.2) "
5555
"before running this migration."
5656
)
5757

5858
return False
5959

6060
def database_forwards(self, app_label, schema_editor, from_state, to_state):
61-
if self.has_teak_release_tables():
61+
if self.has_ulmo_release_tables():
6262
return
6363
return super().database_forwards(app_label, schema_editor, from_state, to_state)
6464

6565
def database_backwards(self, app_label, schema_editor, from_state, to_state):
66-
if self.has_teak_release_tables():
66+
if self.has_ulmo_release_tables():
6767
return
6868
return super().database_backwards(app_label, schema_editor, from_state, to_state)
6969

0 commit comments

Comments
 (0)