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

feat: portable text renderer basert på fremtind.no sitt oppsett #4632

Merged
merged 1 commit into from
Mar 3, 2025

Conversation

kristianulv23
Copy link
Contributor

Tilrettelagt for fleksibel rendering av Portable Text fra Sanity. Etter å ha sammenlignet med oppsettet i fremtind.no, har jeg valgt å gjenbruke deres løsning i stedet for å bygge en ny fra bunnen av.

ISSUES CLOSED: #4631

@fremtind-bot
Copy link
Collaborator

fremtind-bot commented Feb 27, 2025

Forhåndsvisning: https://jokul.fremtind.no/preview/4631-portable-text-renderer/
🔍 Commit: 3da3815

Forhåndsvisningen blir tilgjengelig innen et par minutter. Den fjernes automatisk når pull requesten lukkes.

fremtind-bot added a commit that referenced this pull request Feb 27, 2025
@kristianulv23 kristianulv23 changed the title feat: sett opp portable rext renderer basert på fremtind.no sitt oppsett feat: portable rext renderer basert på fremtind.no sitt oppsett Feb 27, 2025
@kristianulv23 kristianulv23 changed the title feat: portable rext renderer basert på fremtind.no sitt oppsett feat: portable text renderer basert på fremtind.no sitt oppsett Feb 28, 2025
@piofinn
Copy link
Contributor

piofinn commented Feb 28, 2025

Jeg er litt usikker på om jeg har lyst til å dra inn all kompleksiteten fra fremtind.no-implementasjonen med en gang: Vi har hatt en del frustrasjoner med dette, særlig med Zod-parsingen av rik tekst og måten schema-ene er satt sammen der.

Dette er en fin anledning til å se på det på nytt med bakgrunn i erfaringene våre, men det er nok lettere hvis vi ikke drar med oss hele strukturen og modellen.

Edit: For å klargjøre litt så er det spesifikt Zod-schema og tilhørende logikk jeg gjerne vil vente med å ta inn. Vi gikk oss litt vill i komplekse typer og parsing i fremtind.no, på grunn av noen antakelser og grunntanker som ikke nødvendigvis gjelder for dette prosjektet 😊

@kristianulv23 kristianulv23 force-pushed the 4631-portable-text-renderer branch 2 times, most recently from d2cace7 to 083a9a5 Compare February 28, 2025 13:09
fremtind-bot added a commit that referenced this pull request Feb 28, 2025
fremtind-bot added a commit that referenced this pull request Feb 28, 2025
@kristianulv23 kristianulv23 force-pushed the 4631-portable-text-renderer branch from 083a9a5 to a9c12fd Compare February 28, 2025 18:47
fremtind-bot added a commit that referenced this pull request Feb 28, 2025
piofinn
piofinn previously approved these changes Mar 3, 2025
Copy link
Contributor

@piofinn piofinn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Smooth!

Added portable-text related logic

ISSUES CLOSED: #4631
@kristianulv23 kristianulv23 enabled auto-merge March 3, 2025 11:59
@kristianulv23 kristianulv23 added this pull request to the merge queue Mar 3, 2025
fremtind-bot added a commit that referenced this pull request Mar 3, 2025
Merged via the queue into main with commit e900786 Mar 3, 2025
9 checks passed
@kristianulv23 kristianulv23 deleted the 4631-portable-text-renderer branch March 3, 2025 12:17
github-actions bot pushed a commit that referenced this pull request Mar 3, 2025
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.

Implementering av Portable Text Renderer basert på eksisterende beste praksis
3 participants