Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When creating a term with slash "/" in the name, the slash should be escaped when generating term IRI #505

Open
jakubklimek opened this issue Oct 6, 2023 · 2 comments
Assignees
Labels
bug Something isn't working in-dev Řešení implementováno v prostředí DEV

Comments

@jakubklimek
Copy link
Contributor

Slash in IRIs is typically used to separate sections of the IRI path. Therefore, when such slash appears in a term name, it should be escaped, not propagated to the term IRI, so that there is no confusion about the structure of the IRI path such as with:
https://slovník.gov.cz/agendový/104/pojem/technický-název-objektu/subjektu-krátký

@jakubklimek jakubklimek added the bug Something isn't working label Oct 6, 2023
bindeali added a commit that referenced this issue Oct 26, 2023
@bindeali bindeali added this to the Stabilní a výkonný OG milestone Nov 28, 2023
@bindeali
Copy link
Collaborator

bindeali commented Dec 8, 2023

Mělo by být vyřešeno na PROD, prosím o kontrolu.

@jakubklimek
Copy link
Contributor Author

teď se to kóduje jako %2F - otázka je, jestli by se to nemělo např. nahradit pomlčkou. Nemám na to ale konkrétní názor - spíš mi jde o konzistenci escapování/vynechávání. Např & je vynecháno...

@bindeali bindeali removed this from the Stabilní a výkonný OG milestone Feb 2, 2024
bindeali added a commit that referenced this issue Feb 4, 2024
@bindeali bindeali added the in-dev Řešení implementováno v prostředí DEV label Feb 5, 2024
@bindeali bindeali self-assigned this Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working in-dev Řešení implementováno v prostředí DEV
Projects
None yet
Development

No branches or pull requests

2 participants