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

Remove strong from titles #535

Merged
merged 5 commits into from
Oct 25, 2024
Merged

Remove strong from titles #535

merged 5 commits into from
Oct 25, 2024

Conversation

gunnarvelle
Copy link
Member

Fjerner lagra strong-tags fra tittel på artikler.

@gunnarvelle gunnarvelle requested a review from a team October 22, 2024 10:16
@jnatten
Copy link
Contributor

jnatten commented Oct 23, 2024

Burde vi ha noe validering på at dette ikke lagres dersom det ikke skal finnes? 🤔

@gunnarvelle
Copy link
Member Author

Har pr i ed som forhindrer innsetting av strong i tittel.

@jnatten
Copy link
Contributor

jnatten commented Oct 23, 2024

Har pr i ed som forhindrer innsetting av strong i tittel.

Høres ut som vi burde validere det i backenden også?
Vi har allerede tittel-validering i både draft og article så burde vel ikke være stress om det ikke er noe jeg ikke tenker på da? 😄

@gunnarvelle
Copy link
Member Author

Det du meinte ja! Skal sjekke kva eg får til.

@gunnarvelle
Copy link
Member Author

Ser ingen god måte å validere at strong ikkje er i headers uten å endre html-rules ved å legge til blacklist. Dessuten er begrensinga på ikkje strong i headers en spesifikk ndla-greie, og om vi skal tenke at vi skal være åpne for at andre skal kunne bruke løsningene våre er det sikkert dumt å legge til slik begrensning i koden. Vi håndhever uansett dette via slate i ed.

@gunnarvelle gunnarvelle requested review from jnatten and a team October 24, 2024 12:01
@gunnarvelle gunnarvelle merged commit 35e0bc2 into master Oct 25, 2024
9 checks passed
@gunnarvelle gunnarvelle deleted the strip-strong-from-title branch October 25, 2024 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants