-
Notifications
You must be signed in to change notification settings - Fork 2
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
ZON-6179: first stab at the spec for search #23
Conversation
items: | ||
type: array | ||
items: | ||
$ref: "#/components/schemas/CenterpageRegion" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
macht das Sinn, hier wie bei /cp/
eine CenterpageRegion
zurueckzugeben? bei den Suchergebnissen gibt es ja keine unterschiedlichen Regionen - oder ist das dem Umstand geschuldet, dass wir evtl. einfach immer grundsaetzlich region -> area -> items ausliefern wollen?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
letzteres
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Zur Info, prepublic hat sich jetzt doch das Gegenteil gewünscht - nur die Teaser gleich in items
. Siehe #24
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reicht der App ein einfaches "query string"? Oder will die auch die Sortierungs- und Filterungsoptionen haben, die auf www.zeit.de/suche angeboten werden (siehe https://github.com/zeitonline/zeit.web/blob/master/src/zeit/web/site/module/search_form.py)?
@wosc aktuell reicht der App laut Design-Prototyp der einfache query string. Da sind keine Filter vorgesehen... |
👍 |
No description provided.