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

Update related AbstractElements and allow to clear caches after population #64

Merged
merged 15 commits into from Mar 27, 2024

Conversation

rliebi
Copy link
Contributor

@rliebi rliebi commented Mar 14, 2024

Allows to define related AbstractElements per Document and also reindex those.
Allows to clear specific cache tag if population is finished.

@rliebi rliebi requested a review from limenet March 14, 2024 09:26
@rliebi rliebi self-assigned this Mar 14, 2024
@rliebi rliebi added the enhancement New feature or request label Mar 21, 2024
@limenet
Copy link
Member

limenet commented Mar 21, 2024

As discussed:

  • handling cache: fire events instead + add note in README
  • related objects: consider making note in README for projects to handle

@limenet
Copy link
Member

limenet commented Mar 26, 2024

Please re-evaluate the concept of stopping the event propagation; it's necessary but currently it feels like there's duplicated logic.

@limenet limenet merged commit 0e85b98 into main Mar 27, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants