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

ZO-4724: Avoid create article summary #647

Merged
merged 5 commits into from
Mar 14, 2024
Merged

Conversation

AWieczorreck
Copy link
Member

@AWieczorreck AWieczorreck commented Mar 7, 2024

Dieser PR erstellt eine Checkbox im Adminpanel, mit dem man die Artikelzusammenfassung (Summy) für einen Artikel deaktivieren kann.

Ticket: ZO-4724

Checklist

  • Documentation
  • Changelog
  • Tests
  • Translations

gif

@louika louika self-assigned this Mar 11, 2024
Copy link
Contributor

@louika louika left a comment

Choose a reason for hiding this comment

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

Grundsätzlich funktioniert das alles so, aber aus dem Ticket geht hervor, dass dieses Feature für Artikel gedacht ist.
Entsprechend gehört die Property nicht nach ICommonMetadata sondern nach IArticleMetadata hier https://github.com/ZeitOnline/vivi/blob/main/core/src/zeit/content/article/interfaces.py#L28
ne KI-Zusammenfassung für xml config Dateien in vivi seh ich noch nicht kommen 😆 ansonsten fein 👍

Copy link
Contributor

@louika louika left a comment

Choose a reason for hiding this comment

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

works as intended 👍 jetzt nurnoch die entsprechende Payload an den Publisher schicken.
check

@louika louika merged commit 28ed6b5 into main Mar 14, 2024
4 checks passed
@louika louika deleted the ZO-4724_avoid_create_summary branch March 14, 2024 15:39
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