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.

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

Render missing security history events #15702

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jzohrab
Copy link
Contributor

@jzohrab jzohrab commented Apr 2, 2024

Addresses #13934

I've added what I felt was the relevant detail for each event.

@@ -504,6 +504,20 @@ <h2>{% trans %}API tokens{% endtrans %}</h2>
<section id="account-events">
<h2>{% trans %}Security history{% endtrans %}</h2>

{% set role_strings = {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this method is clean but it deviates from existing code, LMK if you don't like it.

@jzohrab jzohrab marked this pull request as ready for review April 2, 2024 15:02
@jzohrab jzohrab requested a review from a team as a code owner April 2, 2024 15:02
warehouse/templates/manage/account.html Outdated Show resolved Hide resolved
warehouse/templates/manage/account.html Outdated Show resolved Hide resolved
@jzohrab
Copy link
Contributor Author

jzohrab commented Apr 3, 2024

Thanks @di for the comment. I've completed this, I think, and removed all of the cruft comments from the PR, and rebased off main. CI now passes.

@jzohrab jzohrab force-pushed the issue_13934_render_missing_security_history_events branch from 5abe94d to 55617c0 Compare April 3, 2024 04:59
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.

None yet

2 participants