Creating a role redirects to the Users page instead of the Roles page
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 78/100
Direzione di ricerca
Leggi app/controllers/kaui/role_definitions_controller.rb, in particolare create e il reindirizzamento esistente di update. Controlla i test funzionali per la creazione dei ruoli e aggiungi un'asserzione che verifichi che il percorso di creazione predefinito reindirizzi a /role_definitions, preservando il comportamento del contesto di ritorno. Il lavoro è completato quando il nuovo test passa e la notifica di successo include il nome del ruolo, come richiesto.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Description
After creating a role from the Roles page, Kaui redirects to the Users page (/admin_allowed_users) instead of back to the Roles list (/role_definitions).
Steps to reproduce
- Open Roles (
/role_definitions) and click Create Role. - Enter a role name and permissions, then click Save Role.
Expected
The role is created and the browser returns to the Roles list, where the new role appears.
Actual
The flash notice "Role was successfully created" is shown on the Users page.
Cause
Kaui::RoleDefinitionsController#create falls back to admin_allowed_users_path when there is no role_return_context in the session:
else
redirect_to admin_allowed_users_path, notice: 'Role was successfully created'
end
update already redirects to role_definitions_path, so create should do the same. The existing redirect back to the user form (when the role was started from the new/edit user screen via role_return_context) is intentional and should stay.
Suggested fix
Redirect to role_definitions_path in the default branch, and include the role name in the notice for consistency with update and the return-context branches. Add a functional test asserting the redirect.
- Lingua principale
- HTML
- Stelle
- 69
- Fork
- 75
- Merge medio
- 2g 14h
- PR unite (30g)
- 5
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di killbill/killbill-admin-ui
-
Payments page should show payment number in a different color so users know it is clickableForse già presa @tungleduyxyz l’ha presa 2 giorni fa. Apertabug
killbill/killbill-admin-ui#679 · 1 assegnatario ·
-
Show invoice amount and balance on the account invoices page without losing paginationForse già presa @tungleduyxyz l’ha presa 2 giorni fa. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
killbill/killbill-admin-ui#678 · 1 commento · 1 assegnatario ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 50/100
killbill/killbill-admin-ui#663 · 1 commento ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 55/100
killbill/killbill-admin-ui#662 ·
-
Kaui 4.0.1 - viewing catalogForse di nuovo libera @tungleduyxyz l’ha presa 354 giorni fa e non c’è nessuna pull request aperta. Apertaenhancement
killbill/killbill-admin-ui#556 · 5 commenti · 1 assegnatario ·
Tutte le issue di killbill/killbill-admin-ui
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
Automattic/node-canvas#2633 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
handsontable/hyperformula#1879 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 79/100
smarty-php/smarty#1220 ·
-
parse_selu lowering loses catastrophic cancellation → silent 0.0 for moderate positive inputsAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
ROCm/AMDMIGraphX#5490 ·
I maintainer di solito rispondono entro 2 giorni