#4138 closed Enhancement (fixed)
admin should be able to change user's notifications
| Reported by: | cash | Owned by: | mrclay |
|---|---|---|---|
| Priority: | normal | Milestone: | Elgg 1.8.7 |
| Component: | Notifications | Version: | Github Master |
| Severity: | minor | Keywords: | |
| Cc: | brett@…, steve@… | Difficulty: |
Description
Change History (7)
comment:1 Changed 16 months ago by cash
- Milestone changed from Elgg 1.8.x to Elgg 1.8.5
comment:2 Changed 11 months ago by mrclay
- Cc steve@… added
comment:3 Changed 11 months ago by mrclay
- Owner set to mrclay
- Status changed from new to assigned
Owner block/settings URLs problem in PR is fixed.
comment:4 Changed 11 months ago by Steve Clay
- Resolution set to fixed
- Status changed from assigned to closed
Fixes #4138: Admin can edit any users' notifications
Changeset: 4a3c49240140449ef4c91c4b999a91b11380db3c
comment:5 Changed 11 months ago by Cash Costello
Merge pull request #280 from mrclay/4138-notifications
Fixes #4138: Admin can edit any users' notifications
Changeset: 264d3643f2c76bedf875e3c586055d9f5ee14132
comment:6 Changed 11 months ago by Steve Clay
Fixes #4138: Admin can edit any users' notifications
Changeset: 4a3c49240140449ef4c91c4b999a91b11380db3c
comment:7 Changed 11 months ago by Cash Costello
Merge pull request #280 from mrclay/4138-notifications
Fixes #4138: Admin can edit any users' notifications
Changeset: 264d3643f2c76bedf875e3c586055d9f5ee14132
Note: See
TracTickets for help on using
tickets.
PR https://github.com/Elgg/Elgg/pull/280
This adds username to the notification settings URLs. It also requires that these pages route through the page handler. There's a remaining problem that, although the page owner is set, the settings URLs lose the username.