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

feat(cli): deploy command including org scope #232

Conversation

BibiSebi
Copy link
Contributor

What?

Since past 2 weeks a new feature has been added to the product where organizations are able to create their own field plugins. For this new feature a new API endpoint has been added so we need to consider it inside our CLI to give the user the option to deploy to an organisation.

Why?

JIRA: EXT-1849

How to test? (optional)

- added logic to be able to add plugins to an organization
- also added text to show user where the plugin can be found inside organization
@BibiSebi BibiSebi requested a review from eunjae-lee July 20, 2023 13:18
@vercel
Copy link

vercel bot commented Jul 20, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
plugin-sandbox ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 20, 2023 1:32pm

@BibiSebi
Copy link
Contributor Author

After merging, the version of CLI still needs to be bumped and a new release created

!accessibleToMyPlugins &&
!accessibleToPartnerPortal &&
!accessibleToOrganizationPortal
) {
console.error(
red('[ERROR]'),
`The token appears to be invalid as it does not have access to either My Plugins or the plugins on the Partner Portal.`,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to include the organization portal here! :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

true :D haha let me quickly add it

Copy link
Contributor

@eunjae-lee eunjae-lee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work !

@BibiSebi BibiSebi merged commit d98108e into main Jul 20, 2023
@BibiSebi BibiSebi deleted the EXT-1849-add-new-option-to-deploy-field-plugins-to-an-organization branch July 20, 2023 13:37
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