-
Notifications
You must be signed in to change notification settings - Fork 0
Kco 32 custom fields #3012
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
base: development3
Are you sure you want to change the base?
Kco 32 custom fields #3012
Conversation
$input['field_type_id'] | ||
); | ||
$customFieldDto = CustomField::from([ | ||
'app' => app(Apps::class), |
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.
@FredPeal use $app
$customFieldModules = CustomFieldsModules::where( | ||
'system_modules_id', | ||
$systemModules->getId() | ||
)->first(); |
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.
@FredPeal firstOrFail no?
$input['field_type_id'] | ||
); | ||
$customFieldDto = CustomField::from([ | ||
'app' => app(Apps::class), |
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.
$app
@@ -40,6 +41,10 @@ extend type Mutation @guard { | |||
@field( | |||
resolver: "App\\GraphQL\\Ecosystem\\Mutations\\CustomFields\\CustomFieldMutation@getAll" | |||
) | |||
createCustomFields(input: CustomFieldInput): CustomField |
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.
should return CustomField!
created_at: DateTime! | ||
updated_at: DateTime! | ||
} | ||
extend type Mutation @guard { |
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.
@FredPeal only super admin can do dthis
) | ||
} | ||
|
||
extend type Query @guard { |
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.
@FredPeal same here
@@ -11,6 +11,7 @@ input CustomFieldInput { | |||
value: Mixed | |||
system_module_uuid: String! | |||
entity_id: ID! | |||
field_type_id: ID |
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.
@FredPeal lets create its own customfield folder
'custom_modules_id' => $this->customFieldEntityValue->customFields->getId(), | ||
'custom_fields_modules_id' => $this->customFieldEntityValue->customFieldsModules->getId(), | ||
'entity_id' => $this->customFieldEntityValue->entity_id, | ||
'value' => $this->customFieldEntityValue->value |
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.
@FredPeal value should be in the first or create no?
|
||
public function execute(): CustomFieldsModules | ||
{ | ||
$customFieldsModules = CustomFieldsModules::getById( |
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.
@FredPeal why not just pass the customfieldmodel
[KCO-576] Integration Activation
[1.x] refactor: import from dev
…on interests and post categories
[development] Graphql shopify
Hotfix email order
Merge pull request #3159 from bakaphp/development
…stsCategoriesAction.php
new refuel order method
refactor: add sku to product variant
…uration in user recommendation tests
…se and API key configuration
…atabase, API key, and region configuration
hotfix: move to exteded search
hotfix: move to exteded search
hotfix: move to exteded search
hotfix: move to exteded search
Add filter for public results in search suggestions
hotfix: move to exteded search
feat: recommendation
hotfix: move to exteded search
Apply fixes from StyleCI
… into KCO-32-custom-fields
…ystem-api into KCO-32-custom-fields
General Description
Please include a summary of the changes and the related issue. Highlight key points of the PR, any relevant information, and reasons for making the change.
Related Issue
A link to the ticket or issue that this PR is related to.(if applicable)
Checklist
A checklist of things that should be done before merging this PR. Linked to the ticket or issue that this PR is related to.(if applicable)
Screenshots (if applicable)
Provide screenshots of impactful changes you have made so that the reviewer can quickly understand the changes.