Oprava odkazu na ical odběr kalendáře - #608
Closed
SmallhillCZ wants to merge 1 commit into
Closed
Conversation
The manual hardcoded https://interni.bosan.cz/api/program/ical, which the rewritten backend no longer served. The feed now lives at /api/public/program/ical, so resolve it from the API root's `_links` (`program:ical`) instead of hardcoding it again, and keep the new URL only as a fallback for when the lookup fails. Refs bosancz/interni-sekce#352
Contributor
|
Takže já nic upravovat nemusím? Prostě odkaz bude stejný a claude upraví stránku, aby se tam zobrazoval vždycky ten správný, ať už bude jakýkoli? (a btw. neuměl by tam udělat i takový to tlačítko na zkopírování, co je třeba u příkazů na diskuzních fórech?) |
Member
Author
|
jo bude tam zatím stejný odkaz, ale ještě to musím trochu doladit uměl |
Member
Author
|
Let's leave the hardcoded path, in future we will remove all the old path2href stuff |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Změny
https://interni.bosan.cz/api/program/icalnatvrdo v šabloně a nová interní ji nepodávala — odkaz vedl do prázdna (Dřív bývala možnost odebírat kalendář akcí přes ical (na adrese… interni-sekce#352)._linkskořene API (program:ical), takže když se endpoint zase někam přesune, návod se srovná sám. Natvrdo zůstává jen nová adresahttps://interni.bosan.cz/api/public/program/icaljako záloha pro případ, že by se lookup nepovedl.Závisí na bosancz/interni-sekce#358, které ten endpoint přidává — do jeho nasazení se použije ta záložní adresa, která začne fungovat ve stejnou chvíli.
Testovací scénář
CTRL + F5⌘ Cmd + ⇧ Shift + Rhttps://interni.bosan.cz/api/public/program/ical(jako text i jako odkaz)..icssoubor s akcemi.Po otestování vždy napiš feedback, buď
Approve review, neboRequest changes. Návod zde: Jak na testování🤖 Generated with Claude Code
https://claude.ai/code/session_01J3H3cr1ZGK3boxU31MM2KJ
Generated by Claude Code