Add CopyMails component#381
Conversation
✅ Deploy Preview for stuyepsilon ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
|
||
| const stuyActivitiesAdminLinks: LinkItem[] = [ | ||
| { to: `${main}/admin/messages`, display: "Messages" }, | ||
| { to: `${main}/admin/copyemails`, display: "Copy Emails" }, |
There was a problem hiding this comment.
Should this be added to (so that Admins of a club see it) or only site Admins?
|
Looking good, some suggestions: For simplicity, could you rename the page and its references to "Roster" instead of "Copy Emails"? Access should be granted to Club&Pub admins, as well as club administrators. The page in the menubar should only be visible if a user is a club admin or Club&Pub admin, like other admin tools (see the user admin checks for the admin tools pages, such as Join Requests, which are not visible if you are browsing the club as a regular member.) The emails that are shown/able to be copied should only be the ones of club members who have club admin access (including creators/presidents/directors of the club). |
There is already option "Roster" visible when as a site admin I click into the club that I am creator of, or admin of. The only time that option is not visible is when I click into the club that I am only member of (when I see only messages). Please clarify if you want me to: a) Add option for site-admins called 'Roster' that would appear in "Admin Tools" section of the menu if user is site-admin and is member of some activity, so that click takes user to the screen where Copy button copies only Admin and Creator emails of that activity, or b) Add option for site admins even for activities where they are creators, or admins, but in that case option cannot be called 'Roster' since it already exists
|
|
Sorry about that- I phrased it poorly I would like the Roster page, which already exists, to also be available to Club&Pub admins, instead of just club admins. Another copy email section should be added to the Roster page with just the emails of club admins/creators/directors. See the image for reference: |
p-predolac
force-pushed
the
fetch_all_email_issue_240
branch
from
bd7b130 to
ef5f683
Compare
|
I redacted real email, but it looks like this |
|
Sorry for the long delay, looks really great! However, the Roster page still isn't visible to Club&Pub sitewide admins, and only to club admins. Could you fix that so both Club&Pub sitewide admins as well as club admins have access to the Roster page? |
2 participants
Added option Copy Emails to Admin Tools and copied the code from roster related to Copying emails. I left members list, so that Admin can see the names corresponding to email.
Please advise if this is what was meant in issue 240