- Issue created by @Pawelgorski87
- 🇨🇦Canada danrod Ottawa
Hi @Pawelgorski87 how's this going? are you going to create a merge request for it?
- Assigned to danrod
- 🇨🇦Canada danrod Ottawa
danrod → changed the visibility of the branch 3432121-move-all-entity to hidden.
- 🇨🇦Canada danrod Ottawa
danrod → changed the visibility of the branch 3432121-move-all-entity to active.
- 🇨🇦Canada danrod Ottawa
The original issue raised by @pawelgorski87 makes sense, there's some URLs / entity links that should be under the admin theme, among others:
/calendar
/calendar/add
/calendar/%/edit
/calendar/all/import
/calendar/event/nid/edit
/calendar/event/nid/deleteThere's also some links can be under the user theme:
/calendar/event/nid
Provided that the user has permissions for that.
I'll look on this today.
- 🇨🇦Canada danrod Ottawa
The issue is that I see no point in having these buttons if you are logged-in as authenticated user and doesn't have privileges to import o edit/create calendars/events:
- 🇨🇦Canada danrod Ottawa
I've moved all the entity management to the admin role, still the user can see generic view /calendar/events, but the regular user can search.
There is an actions column with a "delete" button that doesn't work for non-privileged users (I guess that's another bug to fix):
For privileged users, when editing the calendar events/calendar entities, the admin theme kicks in:
This is the calendar entities edit view:
If you see other places like that needs be switched to the admin status, please let me now, I'll leave this to "Needs Review" for a while
-
danrod →
committed ab154c1b on 3.0.x
Resolve #3432121 "Move all entity"
-
danrod →
committed ab154c1b on 3.0.x
- 🇨🇦Canada danrod Ottawa
This looks ok so far, I'll move it to "Fixed" and let me know if you still see issues with /admin URLs and calendar/event entities
Automatically closed - issue fixed for 2 weeks with no activity.