Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Maintain the list of users in ProxiedUserDetails in chronological (call-time) order #11

Open
billoley opened this issue Apr 5, 2022 · 0 comments
Open

Maintain the list of users in ProxiedUserDetails in chronological (call-time) order #11

billoley opened this issue Apr 5, 2022 · 0 comments

Comments

@billoley
Copy link
Collaborator

billoley commented Apr 5, 2022

Currently, the collection retrieved by ProxiedUserDetails.getProxiedUsers (which are all users) has the final caller as the first entry followed by the entities in the call chain in the order that they happened. This creates some minor complexity in finding the primaryUser and in explaining this order to others.

Modify the places where ProxiedUserDetails iws created such that the proxied entities are first (already in chronological order) and the final caller is last ..... making it also in chronological order
@billoley billoley changed the title Maintain the list of users in ProxiedUserDetails in a more sensible order Maintain the list of users in ProxiedUserDetails in chronological (call-time) order Apr 5, 2022
billoley added a commit that referenced this issue Apr 5, 2022
@billoley
re #11 Maintain the list of users in ProxiedUserDetails in chronologi…
863f151 
…cal (call-time) order
billoley added a commit that referenced this issue Apr 5, 2022
@billoley
re #11 Maintain the list of users in ProxiedUserDetails in chronologi…
4fdb996 
…cal (call-time) order
billoley added a commit that referenced this issue Apr 5, 2022
@billoley
re #11 fix comment, simplify logic in orderProxiedUsers
d3800db
billoley added a commit that referenced this issue Apr 5, 2022
@billoley
re #11 code formatting
6beaef0
billoley added a commit that referenced this issue Apr 5, 2022
@billoley
re #11 Maintain the list of users in ProxiedUserDetails in chronologi…
9204a4a 
…cal (call-time) order (#12)

* re #11 Maintain the list of users in ProxiedUserDetails in chronological (call-time) order

* re #11 fix comment, simplify logic in orderProxiedUsers

* re #11 code formatting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@billoley