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(deps): bump sanity to v2.29.3 and adjust code after the upgrade #26
Conversation
Ser jo greit ut for meg, jeg skjønner ikke 100% hvordan dette henger sammen. Men nå bruker vi v 2.29.3 her, mens vi bruker 2.29.2 i nrkno-sanity? |
Gjør du en test med npm link i nrkno-sanity-repo og testkjøring av bygget på jenkins? 😊 Og gir beskjed når du har gjort det? |
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.
det de andre sa (npm link) og forklaring på hvorfor vi trenger __experimental_search
ellers 👍
@@ -251,6 +251,7 @@ describe('spellcheck-values', () => { | |||
}, | |||
}, | |||
], | |||
__experimental_search: [], |
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.
Trenger vi legge til dette? https://www.sanity.io/docs/studio-search-config
Er det kanskje pga ArraySchemaType?
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.
Feltet er required i ObjectSchemaType
fra Sanity.
@@ -2,7 +2,7 @@ import React, { PropsWithChildren, useCallback, useContext } from 'react'; | |||
import { Box, Button, Card, Flex, Label, Popover, Stack, Tooltip } from '@sanity/ui'; | |||
import { SpellcheckDispatch } from './SpellcheckContext'; | |||
import { SpellcheckProgressbar } from './SpellcheckProgressbar'; | |||
import { TooltipProps } from '@sanity/ui/lib/src/primitives/tooltip/tooltip'; | |||
import { TooltipProps } from '@sanity/ui/lib/dts/primitives/tooltip/tooltip'; |
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.
Kunne vi brukt
import { TooltipProps } from '@sanity/ui/lib/dts/primitives/tooltip/tooltip'; | |
import type { TooltipProps } from '@sanity/ui/lib/dts/primitives/tooltip/tooltip'; |
Her?
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.
Bra forresten 👏 Gjenstår fortsatt testingen ?
6d836d6
to
03c7409
Compare
Scope
bump sanity to v2.29.3 and adjust code after the upgrade.
Reviewers
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.