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

Fixes a problems with reports #739

Merged
merged 3 commits into from
Oct 1, 2024

Conversation

sfisher
Copy link
Contributor

@sfisher sfisher commented Sep 18, 2024

This fixes two problems with the reports for #718 .

Makes it so login is required to view reports and eliminates errors with no user.

It makes it so that changing the date drop-downs for the report lower in the page does not reset the group/realm setting in the upper part of the page.

The reports are not exactly consistent in what they show and I think it's on purpose since they are actually kind of separate reports that have been glommed into one page.

See the explanation in the ticket and we can decide if it's worth reworking them or just make it more clear what each report entails.

The CSV report always includes all items that the user has access to.

The second section (summary) heeds the group/realm settings but gives stats for all time.

The third section with the inline HTML table with stats heeds the group/realm settings and also the dates right above it (and nothing else on the page uses those dates).

If we decide to just make it clearer, I can add explanatory text for the sections and add to the PR.

jsjiang and others added 3 commits September 16, 2024 08:03
@sfisher sfisher linked an issue Sep 18, 2024 that may be closed by this pull request
@sfisher sfisher changed the base branch from main to develop September 18, 2024 23:18
@sfisher sfisher requested a review from jsjiang September 18, 2024 23:20
Copy link
Contributor

@jsjiang jsjiang left a comment

Choose a reason for hiding this comment

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

@sfisher The fixes look good.
Thank you Scott for investigating the differences between the UI and report.

Jing

@sfisher sfisher merged commit 8ca6a59 into develop Oct 1, 2024
1 check passed
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.

Refactor "Download report in CSV format"
2 participants