MidgardVTT-Entwicklung
/
foundry-vtt-system-midgard5
7
0
Fork 0
Code Issues 16 Pull Requests Actions Projects 1 Releases 8 Wiki Activity

changed manifest for easier update #97

Merged
oskaloq merged 7 commits from Manifest-change into develop 2024-04-01 19:30:20 +02:00
Conversation 5 Commits 7 Files Changed 2 +5 -5
oskaloq commented 2024-03-29 17:05:44 +01:00
Member
Copy Link
No description provided.
oskaloq added 1 commit 2024-03-29 17:05:45 +01:00
oskaloq added 1 commit 2024-03-29 17:32:10 +01:00
oskaloq added 1 commit 2024-03-29 18:07:17 +01:00
Byroks approved these changes 2024-03-29 21:12:23 +01:00
Dismissed
oskaloq added 1 commit 2024-03-30 11:57:54 +01:00
oskaloq dismissed Byroks’s review 2024-03-30 11:57:54 +01:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

oskaloq commented 2024-03-30 12:02:55 +01:00
Author
Member
Copy Link

@Byroks:
Ich habe jetzt noch einen token erstellt, read Rechte auf die Releases hat und diesen an die Manifest URLs angehängt:

"manifest": "https://git.byroks.de/MidgardVTT-Entwicklung/foundry-vtt-system-midgard5/releases/download/latest/system.json?token=2455375115b92bc8d87c93df7159eec232456f2d",
"download": "https://git.byroks.de/MidgardVTT-Entwicklung/foundry-vtt-system-midgard5/releases/download/latest/midgard5.zip?token=2455375115b92bc8d87c93df7159eec232456f2d",

Damit brauchst Du jetzt das Projekt für Releases nicht mehr öffentlich zu machen ;->
Das gleiche habe ich auch die Erweiterung getan.

@Byroks: Ich habe jetzt noch einen token erstellt, read Rechte auf die Releases hat und diesen an die Manifest URLs angehängt: ``` "manifest": "https://git.byroks.de/MidgardVTT-Entwicklung/foundry-vtt-system-midgard5/releases/download/latest/system.json?token=2455375115b92bc8d87c93df7159eec232456f2d", "download": "https://git.byroks.de/MidgardVTT-Entwicklung/foundry-vtt-system-midgard5/releases/download/latest/midgard5.zip?token=2455375115b92bc8d87c93df7159eec232456f2d", ``` Damit brauchst Du jetzt das Projekt für Releases nicht mehr öffentlich zu machen ;-> Das gleiche habe ich auch die Erweiterung getan.
Byroks requested changes 2024-03-31 16:28:53 +02:00
Dismissed
Byroks left a comment
Owner
Copy Link

Guter Gedanke, aber zumindest die Versionierung sollte drin bleiben, sonst ist es nicht mehr möglich auf eine spezifische Version zu patchen.

Die Versionierung wird ja hier auch von der gulpfile übernommen, so das man nicht viel machen muss.

Guter Gedanke, aber zumindest die Versionierung sollte drin bleiben, sonst ist es nicht mehr möglich auf eine spezifische Version zu patchen. Die Versionierung wird ja hier auch von der gulpfile übernommen, so das man nicht viel machen muss.
oskaloq commented 2024-03-31 16:55:59 +02:00
Author
Member
Copy Link

Bin jetzt nicht ganz sicher, was Du meinst:
Wenn Du ein neues Release per "npm run package [new version number]" erstellst, wird der "version":"[Neue Version]" Eintrag sowohl in der package.json, wie auch in der system.json weiterhin aktualisiert. Nur die Versionierung in den URLs habe ich auskommentiert. Wenn Foundry dann mitbekommt, dass eine neue Version vorliegt (<- neue Version in der system.json unter der bekannten URL), dann kann es auch gleich das neue Zipfile runterladen und entpacken. Zumindest die URL der Manifest Datei sollte sich also nicht ändern, sonst müsstest Du das System jedesmal über die neue Manifest URL neu installieren, oder?

Bin jetzt nicht ganz sicher, was Du meinst: Wenn Du ein neues Release per "npm run package [new version number]" erstellst, wird der "version":"[Neue Version]" Eintrag sowohl in der package.json, wie auch in der system.json weiterhin aktualisiert. Nur die Versionierung in den URLs habe ich auskommentiert. Wenn Foundry dann mitbekommt, dass eine neue Version vorliegt (<- neue Version in der system.json unter der bekannten URL), dann kann es auch gleich das neue Zipfile runterladen und entpacken. Zumindest die URL der Manifest Datei sollte sich also nicht ändern, sonst müsstest Du das System jedesmal über die neue Manifest URL neu installieren, oder?
Byroks commented 2024-03-31 18:40:13 +02:00
Owner
Copy Link

Zum updaten ja, funktioniert auch super. Das problem allerdings ist, das im manifest latest referenziert wird und nicht mehr die Versions nummer. Dadurch kann nicht mehr auf eine bestimmte version gepatched werden sondern es wird immer die letzte Version genommen. Was problematisch werden kann, falls die letzte version kaputt ist und man zurück patchen muss.

Mir ist das aufgefallen weil ich bei der M5-Erweiterung die Versionen einzeln durch updaten wollte um das mit dem Token zu testen, allerdings wurde bei mir anstelle die gewollte Version, die letzte sofort installiert da dort der manifest link hingezeigt hat.

Zum updaten ja, funktioniert auch super. Das problem allerdings ist, das im manifest latest referenziert wird und nicht mehr die Versions nummer. Dadurch kann nicht mehr auf eine bestimmte version gepatched werden sondern es wird immer die letzte Version genommen. Was problematisch werden kann, falls die letzte version kaputt ist und man zurück patchen muss. Mir ist das aufgefallen weil ich bei der M5-Erweiterung die Versionen einzeln durch updaten wollte um das mit dem Token zu testen, allerdings wurde bei mir anstelle die gewollte Version, die letzte sofort installiert da dort der manifest link hingezeigt hat.
oskaloq commented 2024-04-01 10:28:31 +02:00
Author
Member
Copy Link

Stimmt. Da hast Du Recht. Dazu sollte es aber reichen, wenn der Pfad zur Zip Datei versioniert ist, richtig? Dann ließe sich "manuell" eine spezielle Version installieren, der Automatismus (Standard Update aus Foundry verwendet das Manifest in latest mit der entsprechenden Zip Datei) würde aber trotzdem funktionieren.

Stimmt. Da hast Du Recht. Dazu sollte es aber reichen, wenn der Pfad zur Zip Datei versioniert ist, richtig? Dann ließe sich "manuell" eine spezielle Version installieren, der Automatismus (Standard Update aus Foundry verwendet das Manifest in latest mit der entsprechenden Zip Datei) würde aber trotzdem funktionieren.
oskaloq added 1 commit 2024-04-01 10:57:54 +02:00
oskaloq added 1 commit 2024-04-01 11:14:49 +02:00
oskaloq commented 2024-04-01 11:26:40 +02:00
Author
Member
Copy Link

"npm run package 2.6.1" hat jetzt folgendes Manifest erstellt:

[...]
"version": "2.6.0",
[...]
"manifest": "https://git.byroks.de/MidgardVTT-Entwicklung/foundry-vtt-system-midgard5/releases/download/latest/system.json?token=2455375115b92bc8d87c93df7159eec232456f2d",
	"download": "https://git.byroks.de/MidgardVTT-Entwicklung/foundry-vtt-system-midgard5/releases/download/v2.6.1/midgard5-v2.6.1.zip?token=2455375115b92bc8d87c93df7159eec232456f2d",
[...]
"npm run package 2.6.1" hat jetzt folgendes Manifest erstellt: ``` [...] "version": "2.6.0", [...] "manifest": "https://git.byroks.de/MidgardVTT-Entwicklung/foundry-vtt-system-midgard5/releases/download/latest/system.json?token=2455375115b92bc8d87c93df7159eec232456f2d", "download": "https://git.byroks.de/MidgardVTT-Entwicklung/foundry-vtt-system-midgard5/releases/download/v2.6.1/midgard5-v2.6.1.zip?token=2455375115b92bc8d87c93df7159eec232456f2d", [...] ```
Byroks approved these changes 2024-04-01 17:43:46 +02:00
Byroks left a comment
Owner
Copy Link

Ich bin mir tatsächlich nicht sicher aber wir können es mal probieren, Release erstellen und falls es nicht geht wieder zurück rudern.

Ich bin mir tatsächlich nicht sicher aber wir können es mal probieren, Release erstellen und falls es nicht geht wieder zurück rudern.
oskaloq added 1 commit 2024-04-01 19:30:08 +02:00
oskaloq merged commit c5e5ba199b into develop 2024-04-01 19:30:20 +02:00
oskaloq deleted branch Manifest-change 2024-04-01 19:30:21 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Byroks
Labels
Clear labels
  
bug

Something is not working

  
feature

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
regelfehler

Funktionalität stimmt nicht mit Regeln überein

  
wontfix

This won't be fixed

No Label
bug
feature
help wanted
invalid
question
regelfehler
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: MidgardVTT-Entwicklung/foundry-vtt-system-midgard5#97
No description provided.
Delete Branch "Manifest-change"

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?