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

Id widget refactor #6027

Merged
merged 9 commits into from
May 23, 2024
Merged

Id widget refactor #6027

merged 9 commits into from
May 23, 2024

Conversation

Tishasoumya-02
Copy link
Contributor

Copy link

netlify bot commented May 17, 2024

Deploy Preview for plone-components canceled.

Name Link
🔨 Latest commit 98ca737
🔍 Latest deploy log https://app.netlify.com/sites/plone-components/deploys/664ef06055bf550008012d60

Copy link

netlify bot commented May 17, 2024

Deploy Preview for volto failed. Why did it fail? →

Name Link
🔨 Latest commit 5ab948f
🔍 Latest deploy log https://app.netlify.com/sites/volto/deploys/6646fac136cdfd0008dd550d

Copy link
Sponsor Member

@davisagli davisagli left a comment

Choose a reason for hiding this comment

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

Looks good and works when I tested it. Thanks!

@davisagli
Copy link
Sponsor Member

@sneridagh It's not immediately obvious to me why the storybook build in the generator test is failing

@stevepiercy
Copy link
Collaborator

Should we be using yarn or pnpm to build the storybook? This one uses yarn, whereas we use pnpm for Volto docs when invoked by .readthedocs.yaml.

@davisagli
Copy link
Sponsor Member

@stevepiercy At present, we expect development of volto itself to be done using pnpm, but we still expect yarn for projects built by the volto generator, which is what that test is checking. (On the other hand, the new cookiecutter for volto projects sets up a project using pnpm.)

@stevepiercy
Copy link
Collaborator

stevepiercy commented May 23, 2024

Linkcheck fails because https://www.lanku.eus/ is not reachable. Site is by CodeSyntax, so @erral? I also sent an email to their info@ email.

@stevepiercy
Copy link
Collaborator

@davisagli let's see if it still fails, after updating with main.

@davisagli
Copy link
Sponsor Member

@stevepiercy Well, I assume so, since I updated it within the past hour and only merged one other unrelated PR since then.

@stevepiercy
Copy link
Collaborator

@davisagli does this error mean anything to you? I don't have a clue, and my DuckDuckFu does not lead me anywhere useful.

=> Failed to build the preview
SB_BUILDER-WEBPACK5_0002 (WebpackInvocationError): Module not found: Error: Can't resolve 'prop-types-exact/build/helpers/isPlainObject' in './node_modules/airbnb-prop-types/build/helpers'

@davisagli
Copy link
Sponsor Member

@stevepiercy No, it doesn't. That's why I asked @sneridagh about it above.

@erral
Copy link
Sponsor Member

erral commented May 23, 2024

Linkcheck fails because https://www.lanku.eus/ is not reachable. Site is by CodeSyntax, so @erral? I also sent an email to their info@ email.

site is up again!

@stevepiercy
Copy link
Collaborator

@davisagli LOL they're all green now, even though nothing really changed after @Tishasoumya-02 merged main to this branch.

I found this possible solution:

yarn cache clean

It appears that we use cache in our GitHub Actions, but I don't know whether that is related. For our future selves, we can clear caches from GitHub Workflows, if something totally inexplicable with GitHub Actions recurs again. Reference: https://stackoverflow.com/a/64819132/2214933

@davisagli davisagli merged commit e0bd9cf into main May 23, 2024
70 checks passed
@davisagli davisagli deleted the IdWidgetRefactor branch May 23, 2024 14:20
sneridagh added a commit that referenced this pull request May 30, 2024
* main: (60 commits)
  Object browser icon view v2 (#6054)
  Refactor TextWidget (#6020)
  [client] Remove the custom runner with bare vitest config (#6056)
  [client] Fix login mutation (#6053)
  Fix test script in monorepo root (#6051)
  Fix block chooser search is not focusable when clicked on add button (#5867)
  [@plone/types] BlockExtension as Interface types (#6049)
  Fixed querystring search query type (#6034)
  Id widget refactor (#6027)
  Release 18.0.0-alpha.32
  Release @plone/providers 1.0.0-alpha.1
  Release @plone/components 2.0.0-alpha.9
  Release @plone/client 1.0.0-alpha.15
  Release @plone/types 1.0.0-alpha.13
  Fix @plone/client deps (#6045)
  Allow X-Robots-Tag header in images/files middleware (#6043)
  Fixed showRestricted not applying when rendering a custom BlocksForm (#6026)
  change relative import to alias import (#6041)
  Build documentation and Storybook only when their files change (#6037)
  Use pnpm corepack instead of installing it in CI (#6038)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants