Volto Team Meeting Notes 2021-10-12

tisto · October 12, 2021, 2:59pm

2021-10-12

Attendees

Victor
Timo
Alin
Alex
Tiberiu
David
Katja
Piero

Agenda

Modern Image Scales PLIP (Support for modern image scales (Plone 6) · Issue #3279 · plone/Products.CMFPlone · GitHub)
- What image scales do we want in Plone 6?
- Do we want the same image scales that Plone Classic uses?
- We are not using
  - tile 64:64
  - icon 32:32
  - listing 16:16
- We` are purging the Plone classic
Teaser Block
- Existing implementations
  - kitconcept (part of grid-block)
  - redturtle (2 implementations)
  - eau de web/eea (use kitconcept grid block)
- We want the teaser block in core in the long run
- -> move teaser from @kitconcept/volto-blocks-grid to Core
- -> Variant option (e.g. card) allows to plug in new variants
Grid Block
- Existing implementations
  - kitconcept
    - focus is simplicity, ease of use, re-use existing blocks as subblocks
    - uses array to store “subblocks”
  - redturtle
    - simple, subblock approach
  - eau de web / eea
    - multi-dimensional, re-use engine, higher-level-component, wrapper for block, uses components that are in Volto core
- -> our ideal core block will have the UX from kitconcept with the technical infrastructure of the eea block
Listing block uses fullobjects
- PR Andrea on plone.restapi Fix exclude context in querystringsearch by cekk · Pull Request #1241 · plone/plone.restapi · GitHub
- Remove fullobjects lookup in Volto core, look up brains by default
- image_field points to the field with the image: Add new image field metadata in the catalog by sneridagh · Pull Request #13 · plone/plone.volto · GitHub
- Alternatives
  - add field to metadata
  - add field to a custom serializer
- Define if you want fullobjects for a specific listing variant (wish Piero)
Listing block performance issues (not: query)
- Discussion on the PR: Fix exclude context in querystringsearch by cekk · Pull Request #1241 · plone/plone.restapi · GitHub
- not:query is slow
- not:query is crucial for the UX of the listing block
- removing not:query is not an option because it breaks the batching and result count
- -> find alternative or optimize the not:query
plone.volto
- 3.0.0a2 released (clone of kitconcept.volto)
- 3.1.0a1 released (with a new image_field indexer)
New Seamless mode
- The seamless mode was introduced in Volto 13
- kitconcept was experiencing issues in the demo site and the ploneconf site
  - we worked on this because seamless makes docker deployments easier
  - issues
    - chrome caches the last calls to the site
      - you see json on the cache request
      - hard to overcome
    - hit the back button shows json
    - can lead to cache poisoning when using cloudflare etc. that ignores the vary header
    - request a private image in Volto, the request goes directly to plone and shows a login form
- redturtle put it into production on a small site, no major issues so far
- Re-think seamless mode
  - drop the idea of having frontend and backend under the same path “/”
  - we went back to an implementation to Jens’ proposal in plone.rest to a new traversal (REST traverser ++api++ by jensens · Pull Request #113 · plone/plone.rest · GitHub)
  - reasoning for the switch (Rethink and refactor the seamless mode by sneridagh · Pull Request #2722 · plone/volto · GitHub)
  - we want sensible defaults, so we don’t need to pass lots of configuration options
  - new traversal is always exposing JSON
    - indexers won’t find it
    - more dev friendly (you can go directly to a request)
  - released with plone.rest 2.0.0a1 plone.rest · PyPI
- Deprecate old seamless mode?
  - in the long run, there should be one default way of doing things (both for plone.rest and Volto)
  - “seamless mode” should just become the default and does not need its own way
  - new traversal method should become the default for plone.restapi as well (in the long run)
- we will use the new traversal for the ploneconf site
- kitconcept will start using it and others should do the same
- bake the ++ traversal into Volto as a default
  - allow people to add a configuration option to keep the old behavior
plone.restapi 7.x.x branch
- needs lots of backports
- eau de web is main consumer (Plone 4 sites that run with Volto)
- redturtle has a couple sites, not for Volto
- Timo is happy to hand over responsibilities for plone.restapi 7.x.x, either as “release monkey”, that just does releases or entirely (that would need to be discussed with the release team though)
- we need to make sure plone.restapi 7.x.x does not introduce any breaking changes though (otherwise it would need to become plone.restapi 8 and collide with the 8 branch)
Plone Conference / Fanzone
- Most of the team members will meet in Sorrento!