sh-edraft.de/sh_discord_bot
4
1
Fork 1
Code Issues 20 Pull Requests Actions Packages Projects 3 Releases 27 Wiki Activity

#428 #436

Merged
edraft merged 19 commits from #428 into dev 2023-11-15 16:56:32 +01:00
Conversation 0 Commits 19 Files Changed 114 +2717 -2610
edraft commented 2023-11-13 22:56:08 +01:00
Owner
Copy Link

Ticket Referenz:

#428

Gibt es etwas beim Review zu beachten?

Nein

#### Ticket Referenz: #428 #### Gibt es etwas beim Review zu beachten? Nein
edraft added 15 commits 2023-11-13 22:56:09 +01:00
edraft added 2 commits 2023-11-13 23:38:09 +01:00
edraft changed title from WIP: #428 to #428 2023-11-13 23:38:26 +01:00
edraft added 1 commit 2023-11-14 23:50:08 +01:00
Ebola-Chan approved these changes 2023-11-15 02:21:05 +01:00
Ebola-Chan left a comment
Contributor
Copy Link

Es gibt noch Stellen an denen die Tabelle geändert werden aber im Downgrade werden diese Änderungen nicht mehr Rückgängig gemacht.

Das sind Sachen die mir Aufgefallen sind, ohne dass ich mich groß mit Migration auskenne.

Es gibt noch Stellen an denen die Tabelle geändert werden aber im Downgrade werden diese Änderungen nicht mehr Rückgängig gemacht. Das sind Sachen die mir Aufgefallen sind, ohne dass ich mich groß mit Migration auskenne.
bot/src/bot_data/scripts/1.1.7/2_FixUpdates_up.sql
@ -0,0 +1,195 @@
ALTER TABLE CFG_ServerHistory
Ebola-Chan commented 2023-11-15 01:41:02 +01:00
Contributor
Copy Link

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden
bot/src/bot_data/scripts/1.1.9/1_ShortRoleNameOnlyHighest_up.sql
@ -0,0 +1,134 @@
ALTER TABLE CFG_Server
Ebola-Chan commented 2023-11-15 01:41:28 +01:00
Contributor
Copy Link

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden
bot/src/bot_data/scripts/1.2.0/1_FixUserHistory_up.sql
@ -0,0 +1,45 @@
ALTER TABLE `Users`
Ebola-Chan commented 2023-11-15 01:41:42 +01:00
Contributor
Copy Link

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden
bot/src/bot_data/scripts/1.2.0/2_Birthday_up.sql
@ -0,0 +1,190 @@
ALTER TABLE Users
Ebola-Chan commented 2023-11-15 01:41:58 +01:00
Contributor
Copy Link

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden
bot/src/bot_data/scripts/1.2.0/4_MaxSteamOfferCount_up.sql
@ -0,0 +1,79 @@
ALTER TABLE CFG_Technician
Ebola-Chan commented 2023-11-15 01:42:14 +01:00
Contributor
Copy Link

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden
bot/src/bot_data/scripts/1.2.0/5_MaintenanceMode_up.sql
@ -0,0 +1,84 @@
ALTER TABLE CFG_Technician
Ebola-Chan commented 2023-11-15 01:44:28 +01:00
Contributor
Copy Link

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden
bot/src/bot_data/service/migration_service.py Outdated
@ -31,0 +41,4 @@
if migration.migration_id.endswith("Migration"):
return
self._logger.debug(__name__, f"Migrate old migration {migration.migration_id} to new method")
Ebola-Chan commented 2023-11-15 01:48:11 +01:00
Contributor
Copy Link

Sollte hier nicht vielleicht die Nachricht "Migrate new migration {migration.migration_id} to old method" geloggt werden?

Jedenfalls steht in Zeile 52 die selbe Nachricht drinne.

Sollte hier nicht vielleicht die Nachricht `"Migrate new migration {migration.migration_id} to old method"` geloggt werden? Jedenfalls steht in Zeile 52 die selbe Nachricht drinne.
bot/src/bot_data/service/migration_service.py
@ -31,0 +86,4 @@
if upgrade and version.Version(version_str) > version.Version(bot.__version__):
break
# downgrade run migrations from higher versions
if not upgrade and version.Version(version_str) <= version.Version(bot.__version__):
Ebola-Chan commented 2023-11-15 02:05:15 +01:00
Contributor
Copy Link

Wie ich es verstehe, sammelst du Downgrades von der höchsten Version bis zum Botversion und führst sie an einer anderen Stelle aus. Sollte es da nicht zu einem Fehler führen, da du im Downgrade Tabellen Droppst, die evtl. nicht existieren?

Wie ich es verstehe, sammelst du Downgrades von der höchsten Version bis zum Botversion und führst sie an einer anderen Stelle aus. Sollte es da nicht zu einem Fehler führen, da du im Downgrade Tabellen Droppst, die evtl. nicht existieren?
bot/src/bot_data/service/migration_service.py
@ -31,0 +87,4 @@
break
# downgrade run migrations from higher versions
if not upgrade and version.Version(version_str) <= version.Version(bot.__version__):
continue
Ebola-Chan commented 2023-11-15 02:08:18 +01:00
Contributor
Copy Link

Bei einem Downgrade macht du ein continue wenn die Bedingung zutrifft, während du beim Upgrade ein break abrufst. Gibt es hier einen Fall, in dem die Ordner evtl. nicht in richtiger Reihenfolge stehen und due damit versuchst relevante Ordner abzufangen?

Bei einem Downgrade macht du ein `continue` wenn die Bedingung zutrifft, während du beim Upgrade ein `break` abrufst. Gibt es hier einen Fall, in dem die Ordner evtl. nicht in richtiger Reihenfolge stehen und due damit versuchst relevante Ordner abzufangen?
edraft added 1 commit 2023-11-15 16:56:17 +01:00
edraft merged commit 25df0e4876 into dev 2023-11-15 16:56:32 +01:00
edraft deleted branch #428 2023-11-15 16:56:32 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
Ebola-Chan
Labels
Clear labels   
MVP

   
web-interface
  
bug

Something is not working

  
bugfix

Solution of a problem

  
cli

Changes for command line interface

  
discussion

Something needs to be discussed

  
documentation

Indicates a need for improvements or additions to documentation

  
duplicate

This issue or pull request already exists

  
expense
high

  
expense
low

  
expense
medium

  
expense
very-high

  
help wanted

Need some help

  
improvement

Improvement of a feature

  
invalid

Something is wrong

  
priority
blocker

  
priority
high

  
priority
low

  
priority
medium

  
question

More information is needed

  
refactoring

Code need some refactoring

  
release

Create a release

  
story

New feature

  
wontfix

This won't be fixed

No Label
MVP
web-interface
bug
bugfix
cli
discussion
documentation
duplicate
expense
high
expense
low
expense
medium
expense
very-high
help wanted
improvement
invalid
priority
blocker
priority
high
priority
low
priority
medium
question
refactoring
release
story
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: sh-edraft.de/sh_discord_bot#436
No description provided.
Delete Branch "#428"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?