Skip to content

Create group authorizations from Admin#80

Open
hauptmedia wants to merge 1 commit into
mainfrom
codex/admin-group-authorizations
Open

Create group authorizations from Admin#80
hauptmedia wants to merge 1 commit into
mainfrom
codex/admin-group-authorizations

Conversation

@hauptmedia

Copy link
Copy Markdown

Summary

  • add a user/group selector to the Admin authorization create form
  • send groupId with userId: null when creating group authorizations
  • keep user authorization creation as the default path

Reference

Verification

  • node -e "JSON.parse(require('fs').readFileSync('public/locales/en-US/translation.json','utf8')); JSON.parse(require('fs').readFileSync('public/locales/de-DE/translation.json','utf8')); console.log('locale JSON ok')"
  • npx eslint src/pages/Admin.jsx src/pages/Admin.test.jsx
  • npx vitest run src/pages/Admin.test.jsx
  • npx vitest run --maxWorkers=4
  • npm run build
  • Browser smoke at http://127.0.0.1:5175/admin/authorizations/resource-type/1

@sonarqubecloud

Copy link
Copy Markdown

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.

1 participant