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: debug config available #13

Merged
merged 2 commits into from Feb 4, 2022
Merged

Conversation

snorrees
Copy link
Contributor

@snorrees snorrees commented Feb 2, 2022

NRKNO-4478

Scope

La til ny config-param som gjør at man får en chatty log som forhåpentligvis gjør det enklere å debugge.
Fjernet sjekk på at event har _id til fordel for _rev (som jo faktisk blir validert).

@ajaco har du mulighet til å legge inn dette libbet i nettvideo soonish?

@mollerse tror du denne checklisten i readme og/eller debug logging hadde hjulpet i Åpen gate 🤞 ?
image

Reviewers

Les gjennom oppdatet review, og jsdocs.
Se eventuelt på koden.

Checklist

I have:

  • used conventional commit messages
  • committed with git-hooks active (ran npm install at least once locally before committing)
  • considered if this is a breaking change
  • tested the changes using npm link / yarn link
  • triggered nrkno-sanity-libs-releaser for this branch and checked the status

Branch: NRKNO-4478-debug-iframe-api

  • @nrk/nrkno-iframe-preview-api: 1.1.2 => 1.2.0

When debug: true is provided as a config param,
iframe-preview-api will become chatty in the browser console.
Added an implementation checklist to help with integration.

NRKNO-4478
Copy link
Contributor

@ajaco ajaco left a comment

Choose a reason for hiding this comment

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

Jeg lagde en task for å ta i bruk @nrk/sanity-plugin-nrkno-iframe-preview i nettvideo: https://nrknyemedier.atlassian.net/browse/NRKNO-4483

Tenker jeg gjør den sammen med PR'en som tilgjengeliggjør uttegning av spillelister

packages/nrkno-iframe-preview-api/src/iframe-preview.ts Outdated Show resolved Hide resolved
@mollerse
Copy link
Member

mollerse commented Feb 4, 2022

Vet ikke om det er verdt å nevne det med at _id var et required param i groq'en? Ellers ser oppskrifta ganske identisk ut med det vi måtte gjøre i Åpen Gate.

Den debug featuren ser uansett mad nyttig ut 👍

@snorrees
Copy link
Contributor Author

snorrees commented Feb 4, 2022

Fjernet sjekk på at event har _id til fordel for _rev (som jo faktisk blir validert).

Linje 2 i Pr description ;)
Fjernet sjekk på at event har _id til fordel for _rev (som jo faktisk blir validert).

In fact så sjekkes ikke _rev på iframe messages, bare _eventType som blir syntetsik added av sanity studio på alle events.
_rev valideers når groqQuery er resolved første gang

@snorrees snorrees merged commit d33efdd into master Feb 4, 2022
@snorrees snorrees deleted the NRKNO-4478-debug-iframe-api branch February 4, 2022 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants