Skip to content

BUG: NeosCon schedule api: decoding special chars for topic title, description, speaker summary - #724

Merged
Sebobo merged 5 commits into
mainfrom
bug/neos-con-schedule-special-chars
May 26, 2026
Merged

Sebobo merged 5 commits into
mainfrom
bug/neos-con-schedule-special-chars

Conversation

@minedesign-jl

Copy link
Copy Markdown
Collaborator

Schedule api updated to decode html special characters so a readable string is returned.

goal:

  • & instead of &
  •   should always be
  • title & description, summary etc properly trimmed strings

…cription, speaker name & speaker, summary & event name
@dlubitz

dlubitz commented May 25, 2026

Copy link
Copy Markdown
Member

Thank you @minedesign-jl . From my POV an API shouldn't html encode it's values, so the APU could get used also in a non-HTML context. Rather should the encoding happen on the consuming side. Where did you noticed the wrong encoding? In the NeosCon App?

@minedesign-jl

Copy link
Copy Markdown
Collaborator Author

@dlubitz I am trying to decode, not encode.
afaik the app is currently the only consumer of this api (though not released yet, I think).
With the current live output we get encoded strings for example
"title": "From Web App to Platform: 5+ Years of verein360 (with Neos & Flow)",
which renders in my locally running app like this
image

@dlubitz

dlubitz commented May 25, 2026

Copy link
Copy Markdown
Member

Ahh, sorry. I misread the PR. 👍

@dlubitz dlubitz changed the title bug/NeosCon schedule api: decoding special chars for topic title, description, speaker summary BUG: NeosCon schedule api: decoding special chars for topic title, description, speaker summary May 25, 2026
Comment thread DistributionPackages/Neos.NeosConIo/Classes/Eel/ScheduleHelper.php Outdated
@Sebobo
Sebobo merged commit dc96a77 into main May 26, 2026
3 checks passed
@Sebobo
Sebobo deleted the bug/neos-con-schedule-special-chars branch May 26, 2026 14:35
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Website improvements May 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants