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

New AM user creation does not allow to set roleTenantFilter in the same format as grant/revoke/update methods #512

Open
DenyVeyten opened this issue Oct 28, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@DenyVeyten
Copy link

sfcc-ci Version

2.11.0

NodeJS Version

No response

sfcc-ci Path

No response

Host OS Details

No response

Description

Steps to reproduce:

  1. Call sfcc-ci create AM user method via CLI or API. Pass roles (e.g. with bm-user) and roleTenantFilter with some existing instances
  2. Check created AM user

Actual result: roleTenantFilter does not have specified instances.
Expected result: roleTenantFilter has specified instances.

The issue is with lib/user.js where all methods convert user to the API-compatible shape by calling toInternalUser:

options['body'] = toInternalUser(mergedUser);

while user creation method misses this conversion.

Relevant log output

No response

@DenyVeyten DenyVeyten added the bug Something isn't working label Oct 28, 2023
@DenyVeyten
Copy link
Author

Current workaround: use API-compatible string when calling AM user creation method, e.g. roleTenantFilter: 'ECOM_USER:abcd_dev,abcd_stg'

DenyVeyten added a commit to DenyVeyten/sfcc-ci that referenced this issue Oct 28, 2023
DenyVeyten added a commit to DenyVeyten/sfcc-ci that referenced this issue Oct 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant