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
chore: Initial commit for iframe libs #7
Conversation
NRKNO-4374
NRKNO-4374
NRKNO-4374
NRKNO-4374
NRKNO-4374
NRKNO-4374
NRKNO-4374
NRKNO-4374
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
La til noen kommentarer her og der.
En ting jeg lurer på er hvordan vi skal gjøre endringer på sekvensdiagramet? Er kanskje bedre å comitte noe annet enn en png? Er redd den må bli slettet så snart en ny feature/endring kommer.
Usikker på om jeg kommer til å bruke tid på det. Man skal ha veldig gode grunner til å endre på flyten, pga alle previewene som kjører nå. Protokollen er egentlig låst ned at this point (ie kjørt i prod uten endring i > 1år) |
NRKNO-4374
ty for tilbakemeldingene @AndersCan , har prøvd å rydde litt opp. Forhåpentligvis bedre nå? |
NRKNO-4374
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Brabra 👍
Syntes jeg forstod alt bedre nå :)
|
||
``` | ||
|
||
### About the _rev field |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Da skjønner jeg 👍
Scope: NRKNO-4374
Flytter IFramePreview fra nrkno-sanity til nrkno-sanity-libs.
Flytter et par hooks fra nrkno-sanity til nrkno-sanity-libs (nrkno-sanity-react-utils).
Nytt lib for å integrere render-apps med IFramePreview. Før var dette bare copypasta fra docs, nå er det et lib.
La inn voltaconfig på alt, så da blir alle ting minor-bumped:
Branch: NRKNO-4374-iframe-preview
Reviewers
Les gjennom README.md filene:
Gir det mening?
Checklist
I have:
The branch-build must be manually triggered by the nrk.no team using nrkno-sanity-libs-releaser.
After merging, new version(s) should be published to npm by by triggering
nrkno-sanity-libs-releaser for the master branch.
These manual steps are a safeguard to prevent accidental releases and version changes.