feat: loop level links

[SKairinos]

This change is 

[faucomte97]

Reviewed 7 of 7 files at r1, all commit messages.
Reviewable status: all files reviewed, 3 unresolved discussions (waiting on @SKairinos)

---

game/messages.py line 2435 at r1 (raw file):

    lambda: "Which way to turn? This way or that way?",
    lambda: build_description(title_level115(), "Try to find the shortest route using Python code only. You are getting really good at this!"),
    lambda: "If you are stuck on this one, check that the you are using the Python commands accurately. Click the Py Commands button to help you.",

please remove the unnecessary "the" after "check that"

---

game/migrations/0084_loop_levels.py line 229 at r1 (raw file):

        episode=episode_12,
        path="",
        # next_level=level_123, TODO: connect them when the next levels are enabled.

If we do this it stops the user from loading the next level from the UI but the users could technically still load levels 122 onwards by putting the level number in the URL. Do we want to add something in to prevent this?

---

game/migrations/0084_loop_levels.py line 310 at r1 (raw file):

class Migration(migrations.Migration):
    dependencies = [("game", "0083_add_cows_to_existing_levels")]

You haven't merged in master - the latest PR brings in migrations 84 and 85 so you'll need to rename this one 86 and make it dependent on 85 instead.

[SKairinos]

Reviewable status: 4 of 7 files reviewed, 3 unresolved discussions (waiting on @faucomte97)

---

game/messages.py line 2435 at r1 (raw file):

Previously, faucomte97 (Florian Aucomte) wrote…

please remove the unnecessary "the" after "check that"

Done.

---

game/migrations/0084_loop_levels.py line 229 at r1 (raw file):

Previously, faucomte97 (Florian Aucomte) wrote…

If we do this it stops the user from loading the next level from the UI but the users could technically still load levels 122 onwards by putting the level number in the URL. Do we want to add something in to prevent this?

I think it's fine because it will fail to load the level anyways, letting them know they shouldn't be doing that

---

game/migrations/0084_loop_levels.py line 310 at r1 (raw file):

Previously, faucomte97 (Florian Aucomte) wrote…

You haven't merged in master - the latest PR brings in migrations 84 and 85 so you'll need to rename this one 86 and make it dependent on 85 instead.

Done.

[faucomte97]

Reviewed 3 of 3 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @SKairinos)

[faucomte97]

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @SKairinos)