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

feat: support for subscribing to search queries #463

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Bnyro
Copy link
Contributor

@Bnyro Bnyro commented Oct 30, 2024

  • So ein Feature fände ich persönlich sehr cool um neue Folgen von bestimmten Serien / TV-Sendungen nicht zu verpassen

Beschreibung:

  1. Diese PR fügt einen "Abonnements"-Tab in der Mediathek hinzu
  2. Diese Abonnements sind Such-Queries
  3. Neue Such-Queries können hinzugefügt werden, indem man nach etwas sucht und dann den "Abonnieren"-Button drückt

Es fehlt:

  • Übersicht über die "abonnierten" Such-Queries, wo man diese auch wieder löschen kann
  • Richtige Einbindung des "Abonnieren" / "Deabonnieren" - Buttons in die UI. Der Plan war, diesen Knopf in das Options Menu zu packen, aber ich habe nicht so ganz rausgefunden warum das Menü in dem Suchergebnis-Fragment unsichtbar ist

Bevor ich weiter daran arbeite, wünsche ich mir erstmal Feedback, ob das in die richtige Richtung geht :)

addresses #348

@cemrich
Copy link
Member

cemrich commented Oct 31, 2024

So etwas ist auf jeden Fall geplant (deswegen habe ich die Suche so umfangreich umgebaut), aber ich möchte erst einmal die neue Version veröffentlicht bekommen und die Kinderkrankheiten ausbügeln, bevor neue (große) Features dazu kommen. Ich schaue aber auf jeden Fall mal rein :)

@Bnyro
Copy link
Contributor Author

Bnyro commented Oct 31, 2024

Bisher kann man sich mit der jetzigen PR auch nur den Suchtext merken - es würde vielleicht Sinn ergeben zusätzlich sich noch die Filter wie Dauer und Kanäle zu merken, aber es ist wenigstens schonmal ein erster Prototyp in welche Richtung das gehen könnte.

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