unep-grid / mapx

MapX
https://app.mapx.org
Other
88 stars 24 forks source link

Add email addresses to roles in the metadata panel #1010

Open thomaspiller opened 2 months ago

thomaspiller commented 2 months ago

Roles (i.e., publishers, admins) are displayed in the metadata panel but not user email addresses which are specified in the view edit panel.

Configuration in the view edit panel:

image

Editors listed in the metadata panel:

image

Shouldn't we add email addresses? What do you think?

Originally posted by @thomaspiller in #996

antobenve commented 1 week ago

I think that we should add all addresses if they are specified in the view edit panel.

PierreLacroix commented 1 week ago

I agree with Antonio

fxi commented 1 week ago

We could maybe show a complete list w w/o sections :

Administrators : 
 - a@mail.com
 - b@mail.com
 Publishers : 
 - c@mail.com 
 Temporary :
 - d@mail.com 

Or

 - a@mail.com (Administrator, Last Editor)
- b@mail.com (Administrator)
- c@mail.com (Publisher)
- d@mail.com (Additional)
antobenve commented 1 week ago

I think the issue is not about listing the users that are part of the groups (publisher, admin). It's ok to simply list the groups as it is the case now.

The desired change is about listing the users that are editors of the view while being simple members of the project. Those accounts are not listed in the metadata panel although they are listed in the view configuration panel.

fxi commented 1 week ago

Sure, but since there's a planned refactoring in that module, we could use it as an opportunity to add the described feature. Just an idea. There is a label 'improvement'. We can also label this as a 'bug' and just solve that

antobenve commented 1 week ago

It's a good idea by the way. I thought you didn't get the information clear. Open for the refactoring if agreed.

thomaspiller commented 1 week ago

I also think it's a good idea. It's clearer and more compact.