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

FEATURE: Import custom fields as tags and collections #16

Merged
merged 27 commits into from
Oct 26, 2021

Conversation

pKallert
Copy link
Contributor

@pKallert pKallert commented Oct 18, 2021

This adds a CLI command to import the Canto custom fields as tags and asset collections that are configured accordingly:

./flow canto:importcustomfieldsascollectionsandtags

The command reads the custom fields that should be imported from the configuration (see README.)

Based on #15
See #17

pKallert and others added 25 commits October 5, 2021 10:15
- rename getAllCustomFields() to getCustomFields(), add return type
- rename $tagQuery to $customQueryPart
- rename setTag, getTag() to setActiveTag(), getActiveTag()
- rename setAssetCollection(), getAssetCollection() to
  setActiveAssetCollection(), getActiveAssetCollection()
This renames some variables and fixes code style issues.
# Conflicts:
#	Classes/Service/CantoClient.php
#	Configuration/Settings.yaml
When Canto needs to be accessed e.g. from the CLI, no Neos user is
active and thus no access token can be used. For those cases, a switch
to client credentials mode is now possible.

Note: With this, the need for appId and appSecret in CantoAssetSource
is already back. :)
Using the Canto API from CLI is possible with client credentials mode
by now.
@kdambekalns kdambekalns changed the title Feature: Import Canto customFields as tags and collections FEATURE: Import custom fields as tags and collections Oct 26, 2021
@kdambekalns kdambekalns merged commit 376138d into flownative:main Oct 26, 2021
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