Research-IT-Swiss-TPH / pdf-form-filling-api

API to read, fill and flatten PDF forms.
https://143.198.242.211.sslip.io/
0 stars 2 forks source link

Add 'folders' domain, action and route. #15

Closed tertek closed 7 months ago

tertek commented 8 months ago
tertek commented 8 months ago

@edenst-TPH

Da seit neuster Version des ERD "Projects" in "Folders" umbenannt wurde, muss hier nat¨ürlich auch entsprechend angepasst werden.

tertek commented 7 months ago

Nachdem obige Liste abgeschlossen ist, gilt es wie heute besprochen:

edenst-TPH commented 7 months ago

prep: sync my fork on github (dev branch), pull dev branch to my local dev before we pull to local dev: commit any changes made locally in the feature branch

git status (make shure we are in feature branch)
git add .
git commit -m "added insert and update rules"
git checkout dev
git fetch
git pull

now return to the feature branch and rebase it from the updated dev we just pulled from the server

git checkout add-Folders
git rebase dev

we must commit the feature branch again, the changes made by rebasing. then push it to server

git add .
git commit -m "insert and update rules, rebased from current dev"
git push origin add-Folders

pushes to existing feature branch on server; otherwise we could add and push in one go: git push --set-upstream origin new-branch add-Folders

tertek commented 7 months ago

Zwei Anmerkungen:

  1. Die commit message "features and db migrations of domain add-Folders are afaik complete; testing with http files does not yet work" ist zu lange bzw. beinhaltet Informationen die meiner Meinung nach nicht in den Commit passen. Die commit Nachricht sollte so kurz und objektiv wie möglich sein. Aussagen über das Gesamtsystem sind nicht passend, solang es nicht durch den Commit direkt beeinflusst wird. Deshalb reicht hier als message: "added features and improved DB migrations"

  2. git rebase dev sollte vor dem committen ausgeführt werden, sodass es auch Teil des Commits ist. Dann wird der Fehler error: refused "local is behind" wohl auch nicht angezeigt

edenst-TPH commented 7 months ago
tertek commented 7 months ago

Added in https://github.com/Research-IT-Swiss-TPH/pdf-form-filling-api/pull/19