Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@eguerrant
Copy link
Contributor

@eguerrant eguerrant commented Jun 3, 2025

If a user has direct access and access via one or more roles, it can be unclear if one is granting longer access than the other since the 'Ending' column just shows the max. Added labels on hover to show expiration dates associated with each membership/ownership. Left the date format the same as on other pages (eg. the Access Request list page) to be consistent

Screenshot 2025-06-03 at 11 58 36 AM

@savathoon
Copy link
Contributor

nit - is the ending_date UTC? Could be good to append a timezone to the string to make it clear or localize based on viewer's current TZ

@eguerrant
Copy link
Contributor Author

nit - is the ending_date UTC? Could be good to append a timezone to the string to make it clear or localize based on viewer's current TZ

All the dates shown in Access are in UTC. Fixing that everywhere would be good in the future but for now I feel like being consistent makes more sense

@eguerrant eguerrant merged commit 1f62c78 into main Jun 3, 2025
6 checks passed
@eguerrant eguerrant deleted the chip_date_on_hover branch June 3, 2025 21:59
@somethingnew2-0
Copy link
Collaborator

We could probably use the code in #250 to try and get time zone local timestamps working

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants