kompetenzinventar/ki-backend
5
6
Fork 7
Code Issues Pull Requests 13 Packages Projects Releases 1 Activity

WIP: added script_location entry (it was missed in OSX environment) #3

Closed
gulliver wants to merge 1 commits from alembic_missing_script_location into main
pull from: alembic_missing_script_location
merge into: kompetenzinventar:main
Conversation 5 Commits 1 Files Changed 1 +1 -1
gulliver commented 2021-06-20 14:22:15 +02:00
Owner
Copy Link

Beim Anlegen einer neuen Migration wurde eine Fehlermeldung angegeben, dass der obige Eintrag in der alembic.ini fehlt

Beim Anlegen einer neuen Migration wurde eine Fehlermeldung angegeben, dass der obige Eintrag in der alembic.ini fehlt
gulliver added 1 commit 2021-06-20 14:22:15 +02:00
added script_location entry (it was missed in OSX environment)
All checks were successful
continuous-integration/drone/push Build is passing
Details
continuous-integration/drone/pr Build is passing
Details
f7bb86433b
weeman commented 2021-06-20 16:32:55 +02:00
Owner
Copy Link

Beim Anlegen einer neuen Migration

Wir wollten doch bis Version 1 einfach die erste Migration benutzen. Also einfach aus den aktuellen Modellen neu generieren.

Zum Fehler von dir: Wie hast du denn versucht die Migration anzulegen?

> Beim Anlegen einer neuen Migration Wir wollten doch bis Version 1 einfach die erste Migration benutzen. Also einfach aus den aktuellen Modellen neu generieren. Zum Fehler von dir: Wie hast du denn versucht die Migration anzulegen?
weeman requested review from Owners 2021-06-20 19:29:13 +02:00
weeman removed review request for Owners 2021-06-20 19:29:22 +02:00
weeman changed title from added script_location entry (it was missed in OSX environment) to WIP: added script_location entry (it was missed in OSX environment) 2021-06-20 19:31:01 +02:00
gulliver commented 2021-06-24 12:35:25 +02:00
Author
Owner
Copy Link

alembic revision -m "split user to user and profile"

alembic revision -m "split user to user and profile"
weeman commented 2021-06-24 12:42:07 +02:00
Owner
Copy Link

alembic revision -m "split user to user and profile"

Okay. Für das Projekt hier wird alembic net direkt aufgerufen sondern als Flask Kommando. Dann sind auch alle Sachen richtig gesetzt.

Siehe: https://flask-migrate.readthedocs.io/en/latest/

Ich würde dann hier den PR gerne einfach schließen und später dann einen Hinweis in die README.md aufnehmen wenn es okay ist @gulliver ?

> alembic revision -m "split user to user and profile" Okay. Für das Projekt hier wird `alembic` net direkt aufgerufen sondern als Flask Kommando. Dann sind auch alle Sachen richtig gesetzt. Siehe: https://flask-migrate.readthedocs.io/en/latest/ Ich würde dann hier den PR gerne einfach schließen und später dann einen Hinweis in die `README.md` aufnehmen wenn es okay ist @gulliver ?
gulliver commented 2021-06-24 12:50:02 +02:00
Author
Owner
Copy Link

Für mich Ok.

Für mich Ok.
👍 1
weeman commented 2021-06-26 09:49:38 +02:00
Owner
Copy Link

Ich habs hier reingeschrieben: https://git.wtf-eg.de/kompetenzinventar/ki-backend#alembic-befehle

Probier mal aus, ob das so klappt. Falls net müssen wir doch was machen. Aber ich denke mit flask db ... sollte er alles richtig setzen.

Ich habs hier reingeschrieben: https://git.wtf-eg.de/kompetenzinventar/ki-backend#alembic-befehle Probier mal aus, ob das so klappt. Falls net müssen wir doch was machen. Aber ich denke mit `flask db ...` sollte er alles richtig setzen.
weeman closed this pull request 2021-06-26 09:49:38 +02:00
All checks were successful
continuous-integration/drone/push Build is passing
Details
continuous-integration/drone/pr Build is passing
Required
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something is not working

  
doing

   
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
ready

Bereit für Release

  
review

   
security

   
wontfix

This won't be fixed

No Label
bug
doing
duplicate
enhancement
help wanted
invalid
question
ready
review
security
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: kompetenzinventar/ki-backend#3
No description provided.
Delete Branch "alembic_missing_script_location"

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?