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

#2566 Fixed create new user - added spring-security config for /api/u… #2567

Merged
merged 1 commit into from
Jun 30, 2023

Conversation

Limraj
Copy link
Collaborator

@Limraj Limraj commented Jun 15, 2023

…sers/validate only admin; change int type on String for Guard.hasUserOwnerPermission; change log level from info to debug in UsersAPI - reduction logs;

…sers/validate only admin; change int type on String for Guard.hasUserOwnerPermission; change log level from info to debug in UsersAPI - reduction logs;
@Limraj Limraj added this to the 2.7.5.4 milestone Jun 15, 2023
@github-actions
Copy link

Java Script Mocha Unit Test Results

268 tests  ±0   268 ✔️ ±0   5s ⏱️ ±0s
  70 suites ±0       0 💤 ±0 
    1 files   ±0       0 ±0 

Results for commit bbd3fcd. ± Comparison against base commit 7ff86fa.

@github-actions
Copy link

Java JUnit Test Results

2 083 tests  ±0   2 083 ✔️ ±0   36s ⏱️ +10s
     90 suites ±0          0 💤 ±0 
     90 files   ±0          0 ±0 

Results for commit bbd3fcd. ± Comparison against base commit 7ff86fa.

@Limraj Limraj requested a review from Patrykb0802 June 15, 2023 18:25
@Patrykb0802 Patrykb0802 self-requested a review June 30, 2023 11:25
Copy link
Contributor

@Patrykb0802 Patrykb0802 left a comment

Choose a reason for hiding this comment

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

Tested:
-Created new user on new UI as admin, no error/warning in console

@Limraj Limraj merged commit c0a2df2 into release/2.7.5.4 Jun 30, 2023
10 checks passed
This pull request was closed.
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