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

Add Vite (client only, no SSR) build. Update Next.js 14.2.2 and Remix to 2.8.0 #5970

Merged
merged 17 commits into from
Apr 25, 2024

Conversation

sneridagh
Copy link
Member

No description provided.

* main:
  Improvements to the monorepo setup with utilities, especially ESLint.… (#5969)
  DEV: put nvm installation section into a separate include file (#5968)
Copy link

netlify bot commented Apr 20, 2024

Deploy Preview for volto canceled.

Name Link
🔨 Latest commit c1ae7d2
🔍 Latest deploy log https://app.netlify.com/sites/volto/deploys/662a0cc444dbbc0008724e3c

Copy link

netlify bot commented Apr 20, 2024

Deploy Preview for plone-components canceled.

Name Link
🔨 Latest commit c1ae7d2
🔍 Latest deploy log https://app.netlify.com/sites/plone-components/deploys/662a0cc46f2021000808ab80

Copy link
Collaborator

@stevepiercy stevepiercy left a comment

Choose a reason for hiding this comment

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

Docs LGTM. Thank you!

@sneridagh
Copy link
Member Author

@stevepiercy I added some more docs to the core developer section, could you please do a quick check?

@sneridagh sneridagh requested a review from stevepiercy April 24, 2024 09:01
Copy link
Collaborator

@stevepiercy stevepiercy left a comment

Choose a reason for hiding this comment

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

Mostly nitpicky stuff, but one question needs verification regarding Yeoman usage in the generator.

@@ -289,21 +289,59 @@ Used by Volto, you can also use it in other JavaScript frameworks and environmen
## Supported frameworks

Plone supports several frontend implementations, the main one being Volto as the default frontend and reference React-based implementation.
Copy link
Collaborator

@stevepiercy stevepiercy Apr 24, 2024

Choose a reason for hiding this comment

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

Suggested change
Plone supports several frontend implementations, the main one being Volto as the default frontend and reference React-based implementation.
Plone supports several frontend {term}`reference implementation`s.
Volto, a React-based reference implementation, is the default frontend for Plone 6.

Copy link
Member Author

@sneridagh sneridagh Apr 24, 2024

Choose a reason for hiding this comment

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

@stevepiercy I'd like to start using the "reference implementation" concept here and there. We talked about it during the sprint, if you think that the wording is not correct and we should find a better one, let's do it, but I'd like to convey this from now on for Volto and Classic.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Oh, I thought it was a mistake, not intentional. In that case the term is "reference implementation" and should not have other terms inserted within. I updated my suggestion accordingly.

I added the term to the Documentation glossary in plone/documentation#1656.

docs/source/contributing/developing-core.md Outdated Show resolved Hide resolved
docs/source/contributing/developing-core.md Outdated Show resolved Hide resolved
docs/source/contributing/developing-core.md Outdated Show resolved Hide resolved
The others are currently under heavy development, marked as experimental and for now, they remain as a proof of concept.
Although they do work now in an acceptable way, the implementation might change in the future.
Please note that these implementations only show how to access the public Plone content in the current site and dealing with data fetching and routing.
They use the Plone Frontend Strategic Packages (`@plone/registry`, `@plone/client`, `@plone/components`, etc).

### Plone

The default frontend and reference React-based implementation in Plone is Volto.
Copy link
Collaborator

@stevepiercy stevepiercy Apr 24, 2024

Choose a reason for hiding this comment

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

Suggested change
The default frontend and reference React-based implementation in Plone is Volto.
Volto is the default frontend for Plone 6.
It is a React-based {term}`reference implementation`.

Copy link
Member Author

Choose a reason for hiding this comment

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

Same as above

Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggestion updated.

docs/source/contributing/developing-core.md Outdated Show resolved Hide resolved
docs/source/contributing/developing-core.md Outdated Show resolved Hide resolved
docs/source/contributing/developing-core.md Outdated Show resolved Hide resolved
docs/source/contributing/developing-core.md Outdated Show resolved Hide resolved
docs/source/contributing/developing-core.md Outdated Show resolved Hide resolved
@sneridagh
Copy link
Member Author

@stevepiercy please take a look at my comments above.

Copy link
Collaborator

@stevepiercy stevepiercy left a comment

Choose a reason for hiding this comment

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

Ready to merge, pending my updated suggestions.

@stevepiercy
Copy link
Collaborator

If the readme-link-check build fails, it is due to a temporary outage of https://plone.org.br/ See https://github.com/plone/volto/actions/runs/8828868089/job/24238689105#step:4:143

@sneridagh sneridagh merged commit 5d4338f into main Apr 25, 2024
73 of 74 checks passed
@sneridagh sneridagh deleted the updatedepspocs branch April 25, 2024 08:34
sneridagh added a commit that referenced this pull request Apr 29, 2024
* main: (49 commits)
  Update to Plone 6.0.11 (#5989)
  Defines the last 4 parameters of the `asyncConnect` function with optional (#5986)
  Flexibilize the pins for all ESlint deps, in Volto and generators (#5991)
  Release 18.0.0-alpha.29
  Release @plone/types 1.0.0-alpha.11
  fix: pass down locale to IntlProvider (#5976)
  Add Vite (client only, no SSR) build. Update Next.js 14.2.2 and Remix to 2.8.0 (#5970)
  Fix no router link in logo (#5981)
  Improve postinstall script, building all the deps (#5980)
  Better BlocksData types (#5979)
  Add missing types ts fields
  Release 18.0.0-alpha.28
  Release @plone/slate 18.0.0-alpha.12
  Release @plone/registry 1.5.6
  Improvements to the monorepo setup with utilities, especially ESLint.… (#5969)
  DEV: put nvm installation section into a separate include file (#5968)
  Bundle optimization (#5295)
  [client] Move provider to own package - Use parcel - `@plone/providers` (#5887)
  Fix flaky test 'As editor I can add links' by using getSlateEditorAndType (#5966)
  Fix rendering if ConditionalLink has no children (#5963)
  ...
sneridagh added a commit that referenced this pull request May 7, 2024
* main: (1206 commits)
  Allow Makefile to be modified by Makefile.local if present (#5997)
  Cypress replaced wait with assertions (#5998)
  Replace spinner with Progress bar and fix it's position . (#5620) (#5929)
  Fix type for component registry components (#6002)
  Fix 301 and 302 redirects. (#5999)
  Release 18.0.0-alpha.30
  Force add .d.ts files to releaser
  Release generate-volto 9.0.0-alpha.15
  Fix server side sidebar rendering (#5993)
  Cleaned up useless injectIntl in DefaultView (#5995)
  Fix image disappears after pressing the Enter key on title field in i… (#5975)
  Update to Plone 6.0.11 (#5989)
  Defines the last 4 parameters of the `asyncConnect` function with optional (#5986)
  Flexibilize the pins for all ESlint deps, in Volto and generators (#5991)
  Release 18.0.0-alpha.29
  Release @plone/types 1.0.0-alpha.11
  fix: pass down locale to IntlProvider (#5976)
  Add Vite (client only, no SSR) build. Update Next.js 14.2.2 and Remix to 2.8.0 (#5970)
  Fix no router link in logo (#5981)
  Improve postinstall script, building all the deps (#5980)
  ...
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.

2 participants