Skip to content

feat: add users and spaces to grantToken, allow userId in configuration#282

Merged
are merged 5 commits into
masterfrom
feat/CLEN-562
Jul 5, 2022
Merged

feat: add users and spaces to grantToken, allow userId in configuration#282
are merged 5 commits into
masterfrom
feat/CLEN-562

Conversation

@are

@are are commented Jun 28, 2022

Copy link
Copy Markdown
Contributor

feat: add users and spaces to grantToken

Allows to specify users and spaces in grantToken method.

feat: allow userId in configuration

Allows to use userId instead of uuid in configuration.

return operationConstants.PNAccessManagerGrantToken;
}

function hasVspTerms(incomingParams) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I see VSP word in real life!!! :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I wasn't sure if I should use this to be honest, but couldn't figure out anything else :P


},
groups: {
it('should throw when mixing legacy and VSP terms', async () => {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍🏻

@are

are commented Jul 1, 2022

Copy link
Copy Markdown
Contributor Author

@client-engineering-bot release as v7.2.0

@are are requested review from kazydek and techwritermat as code owners July 1, 2022 11:32
@are are merged commit 0a86870 into master Jul 5, 2022
@are are deleted the feat/CLEN-562 branch July 5, 2022 15:15
@github-actions

github-actions Bot commented Jul 5, 2022

Copy link
Copy Markdown

🚀 Release successfully completed 🚀

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.

3 participants