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

Initial proof of concept #1

Open
wants to merge 25 commits into
base: main
Choose a base branch
from
Open

Initial proof of concept #1

wants to merge 25 commits into from

Conversation

instification
Copy link
Member

@instification instification commented Nov 8, 2023

Main concepts/tasks:

  • Register dynamic vocabularies as persistent utilities
  • Subclass AjaxSelectWidget to add supermodel attribute support
  • Ensure clean uninstall
  • Create plone.app.querystring registry entries (for volto search block query/facet support)
  • Create a readme

Copy link
Member

@djay djay left a comment

Choose a reason for hiding this comment

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

the more I think about it, indexvocabularies doesn't really make sense.
What it does is either uniquevaluesvocabularies or addabletagvocabularies but I think dynamicvocabularies is the best name.

README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
Copy link
Member

@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.

Some grammar, syntax, and style suggestions. Otherwise looks good. Thank you!

README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
Depending on your specific usecase you might be able to try the following:

- Create custom behaviors in your own addon
- Use the `plone.app.vocabularies.Catalog` vocabualary. See `this discussion <https://community.plone.org/t/widget-parameter-for-catalogsource-based-choicefield/18129/3>`_
Copy link
Member

Choose a reason for hiding this comment

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

reStructuredText uses double backticks for inline literals, whereas Markdown uses single backticks.

Suggested change
- Use the `plone.app.vocabularies.Catalog` vocabualary. See `this discussion <https://community.plone.org/t/widget-parameter-for-catalogsource-based-choicefield/18129/3>`_
- Use the ``plone.app.vocabularies.Catalog`` vocabualary. See `this discussion <https://community.plone.org/t/widget-parameter-for-catalogsource-based-choicefield/18129/3>`_

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.

3 participants