Skip to content

Conversation

sneridagh
Copy link
Member

@sneridagh sneridagh commented Oct 4, 2025


📚 Documentation preview 📚: https://volto--7447.org.readthedocs.build/

@sneridagh sneridagh changed the title Tutorial, add a likes button [Seven] Tutorial, add a likes button Oct 4, 2025
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.

This looks really good. Most of my suggestions are for the nitpicky s/addon/add-on. The most important one is to use text instead of images of text.

"keywords": "Plone, Project Title, tutorials"
---

# Tutorials
Copy link
Collaborator

Choose a reason for hiding this comment

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

YESSSSSS!

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.

This looks really good. Most of my suggestions are for the nitpicky s/addon/add-on. The most important one is to use text instead of images of text.

sneridagh and others added 2 commits October 6, 2025 10:18
Co-authored-by: Steve Piercy <web@stevepiercy.com>
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.

Two warnings need fixing. I'm not sure if the emoji one can be fixed.

I found a few more addon-name stragglers, too.

# External Prisma database support for a Seven add-on

This tutorial will show you how to add a small "Like" counter button to any Seven content object.
It will persist its value in an external database using {term}`Prisma`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

This needs a definition in plone/documentation's glossary before it can be merged. See https://app.readthedocs.org/projects/volto/builds/29831361/#288836463--290.

What is Prisma? I honestly tried to find a definition on Prisma's site, and I fell into a deep rabbit hole of Prisma this-and-that.

Copy link
Member Author

Choose a reason for hiding this comment

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

sneridagh and others added 2 commits October 6, 2025 10:58
Co-authored-by: Steve Piercy <web@stevepiercy.com>
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.

Somewhere in the rush to publish, we missed this bad reference.

https://github.com/plone/volto/actions/runs/18275779783/job/52027479172?pr=7447#step:7:84

* seven:
  Monorepodocs (#7441)
  [Seven] Pretty Tailwind  (#7434)
  [Seven] Adding Tailwind to your addon (#7448)
  [Seven] Added loading of rootLoaderData utilities in the root route (#7449)
  [Seven] Added registry server loader feature (#7443)
  Fix providers missing from ESlint config
  [Seven] Remove all unused imports, due to missing rule (#7395)
@sneridagh
Copy link
Member Author

@stevepiercy fixed.

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.

Pending the build and deploy of plone/documentation so that the glossary term shows up. We'll need to do another build to check https://github.com/plone/volto/actions/runs/18276343014/job/52029237083#step:7:88 is resolved.

@sneridagh
Copy link
Member Author

@stevepiercy merging, the build looks good now, the glossary term is in place.

@sneridagh sneridagh merged commit 9034075 into seven Oct 6, 2025
28 checks passed
@sneridagh sneridagh deleted the documentPrisma branch October 6, 2025 12:35
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.

2 participants