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

GenericItem benötigt noch Referenzen zu den vorhandenen Haupt-Datentypen #192

Open
donni106 opened this issue Feb 19, 2021 · 7 comments · Fixed by #190
Open

GenericItem benötigt noch Referenzen zu den vorhandenen Haupt-Datentypen #192

donni106 opened this issue Feb 19, 2021 · 7 comments · Fixed by #190
Assignees
Labels
enhancement New feature or request

Comments

@donni106
Copy link
Member

genauso wie ein generic item weitere generic_items haben kann, denke ich wäre es sinnvoll, wenn es ebenso news_items, event_records, points_of_interest und tours geben würde.
das wurde in #190 noch nicht berücksichtigt.

@donni106 donni106 added the enhancement New feature or request label Feb 19, 2021
@donni106 donni106 linked a pull request Feb 19, 2021 that will close this issue
@marcometz
Copy link
Member

ja, dazu müssen wir :

  • die oben genannten Datentypen erweitern, damit diese polymorphisch wiederverwendbar sind
  • für die oben genannten Datentypen InputTypes definieren
  • has_many relations in GenericItem definieren mit den nested_accessors

Wie bei Mutations::CreateGenericItem doppeln sich dann die argument Definitionen zum zugehörigen Types::InputTypes::GenericItemInput.
Es wäre als Schritt 1 also extrem sinnvoll herauszubekommen, wie die InputTypes in den Mutations importiert werden können, da wir sonst eine Menge komplexen DuplicateCode haben werden.

@marcometz
Copy link
Member

wenn man einen GenericItem anlegt und dieser mit einem oder mehreren NewsItems verknüpft ist, so würden diese NewsItems dann aber auch direkt erreichbar sein über die bisherigen GraphQL Endpunkte NewsItems. Ich weiß nicht, ob das so sinnvoll ist.

@donni106
Copy link
Member Author

[...] so würden diese NewsItems dann aber auch direkt erreichbar sein über die bisherigen GraphQL Endpunkte NewsItems [...]

Verstehe nicht so richtig, was daran neu oder anders wär? NewsItems sind über GraphQL Endpunkt für NewsItems erreichbar, ja!?

@marcometz
Copy link
Member

Ich lege ein GenericItem namens "Job" an und darin enthalten ist ein NewsItem "Der Job ist total toll", dann habe ich in der Liste der aktuellen News auch ein NewsItem "Der Job ist total toll".

@marcometz
Copy link
Member

Ich lege ein GenericItem "Job" an und schreibe das ich einen Maler suche an dem Standort "points_of_interest bei mir zu Hause". Dann habe ich in der Liste aller POI auch den Eintrag "bei mir zu Hause"

@donni106
Copy link
Member Author

donni106 commented Mar 10, 2021

Danke. Verstehe ich jetzt besser, kann ich nicht eindeutig beantworten. Sollte also was fürs morgige Meeting sein.

@donni106
Copy link
Member Author

Das hätte nicht geclosed werden sollen mit dem Merge von #190, vermute ich.

@donni106 donni106 reopened this Mar 11, 2021
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 a pull request may close this issue.

4 participants