- Issue created by @jhuebsch
Thanks to
git log -S
there is a pointer to the original issue, which is #658148: The "access user profiles" permission is poorly named โ . Have the conditions that led to the current naming changed? It has been fifteen years.- ๐ฎ๐ณIndia Sandeep Sanwale
Sandeep Sanwale โ made their first commit to this issueโs fork.
- Merge request !7014Issue #3427427: "Access User Profiles" permission label is misleading โ (Open) created by Sandeep Sanwale
- Status changed to Needs review
11 months ago 5:19am 13 March 2024 - ๐ฎ๐ณIndia Sandeep Sanwale
changed the label of access user profiles from View user information to access user profiles in user.permissions.yml .
- Status changed to Needs work
11 months ago 1:36pm 13 March 2024 The issue summary must be edited to include why this reversal of a prior decision is appropriate.
- ๐บ๐ธUnited States jhuebsch
I have updated the Issue summary with a response to the related issue as well as an alternative proposed resolution of including a description for this permission.
- Status changed to Needs review
10 months ago 4:16am 19 March 2024 - ๐ฎ๐ณIndia Sandeep Sanwale
I have added the description for the Access User Profiles permission . please review this .
- Status changed to Needs work
10 months ago 1:56pm 19 March 2024 - ๐บ๐ธUnited States smustgrave
Now the solution goes against the issue summary. Proposed solution should match
- ๐ฎ๐ณIndia Binoli Lalani Gujarat
Binoli Lalani โ made their first commit to this issueโs fork.
- Status changed to Needs review
10 months ago 5:21pm 26 March 2024 - ๐ฎ๐ณIndia Binoli Lalani Gujarat
Hello,
I updated the permission description as per Proposed resolution and fixed unit testcase errors. Please review latest commit.
Thank you
- Status changed to Needs work
10 months ago 5:32pm 26 March 2024 - ๐บ๐ธUnited States smustgrave
Left a comment but if read out loud doesn't sound correct. "With this permission can do this. And without this permission you can't do this" is kinda how I'm reading this. Left a comment
Still appears to have test failures.
- ๐ฉ๐ชGermany demonde
It would be better to call this "Access Any User Profile" instead of "Access User Profiles", so it is clear this does not mean that users can access their own profile.
- First commit to issue fork.
- ๐ฎ๐ณIndia sakthi_dev
Updated the title/name of the permission. Also updated the IS.
- First commit to issue fork.
- ๐ณ๐ฟNew Zealand quietone
Changes are made on on 11.x (our main development branch) first, and are then back ported as needed according to our policies.