Add new data_resource GenericItem #190
Conversation
The model GenericItem has all features of all other data_resources. - Includes a store for a JSON payload - includes filter method by role - has an ancresty tree to nest other gerneric_items
|
ich denke ich werde die verschachtelten Struckturen wieder mit dem has_ancestry Gem umsetzen. |
Move all graphqler query types in an subfolder and namespace all query types
Move all graphql input types in an subfolder and namespace all input types
- graphql query zum Abrufen verschachtelter Nodes
- graphql Mutation zum anlegen verschachtelter Nodes
Beispiel:
`mutation {
createGenericItem(
author: ""
genericType: "Job"
title: "JobAngebot 1"
genericItems: [{title: "Job 2", genericType: "Offer", genericItems: [{title: "C", genericType: "Search" }]}]
webUrls: ….`
Beispiel:
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
uninitialized constant GenericItem::Company
mutation {
createGenericItem(
author: ""
genericType: "Job"
payload: { foo: "bar" }
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
heftig. aber geil. so in etwas hab ichs mir auch vorgestellt.
die graphql/json schema dateien hätte ich hier noch vermutet? brauchen wir die doch nicht wirklich? rails graphql:schema:dump
viel spaß mit den (22) comments :D
codeclimate wär bitte auch nochmal durchzugehen, um zu gucken, was davon direkt angewendet werden könnte.
| def unique_id | ||
| return external_id if external_id.present? | ||
|
|
||
| fields = [title, published_at] |
There was a problem hiding this comment.
hier vllt auch wie bei news, wenn es keinen title gibt, title = content_blocks.first.try(:title)?
There was a problem hiding this comment.
hmm... wir haben ja ein explizites Attribute title am Model GenericTitle, was bei bei News "eigentlich" nicht haben.
Zumindest greift der unique_id bei news nicht auf das direkte Attribute zurück. Ich glaube hier ist was bei News komisch. Wird dort das direkte Attribute überhaupt verwendet?
There was a problem hiding this comment.
kann ich nicht sagen. im CMS, wenn eine nachricht angelegt wird, kann man title auch nicht direkt eintragen.
Filter GenericItems per user and role permissions
Assign GenericItem to one or many categories
Alle im Model definierten has_many Relationen sind nun über GraphQL erreichbar und erstellbar
Inkl. GraphQL API Anpassungen
Sortierung der Methoden sofern möglich
operating_company ist bereits eindeutig
Damit im CMS die Syntax zu den anderen Forms identisch bleiben kann
donni106
added
enhancement
GenericItems lassen sich nun aufsteigend und absteigend nach dem Feld publication_date sortiert ausgeben: - publicationDate_ASC - publicationDate_DESC
This comment has been minimized.
This comment has been minimized.
Die möglichen Sortieroptionen werden nun in allen Resolvern alphabetisch aufgelistet.
|
Code Climate has analyzed commit 72fbd5b and detected 317 issues on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
The model GenericItem has all features of all other data_resources.