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

Reports counter is for the last 30 days by default #91

Open
williamdes opened this issue Jul 18, 2023 · 6 comments
Open

Reports counter is for the last 30 days by default #91

williamdes opened this issue Jul 18, 2023 · 6 comments

Comments

@williamdes
Copy link
Contributor

When the Month filter is "any" and a domain is selected it felt wrong to see the count for the last 30 days.
Can you add a setting for it to count all reports or another time period ?

Or add some header to the count block to say it's for the last 30 days, so the user does not need to hover the counts to figure out the counts are for the last 30 days ?

On this screenshot taken from : https://aws.amazon.com/blogs/mt/understanding-aws-lambda-behavior-using-amazon-cloudwatch-logs-insights/
image

Amazon has a nice selector that shows that the time period is 30 days. Maybe we could have something like that ?

@liuch
Copy link
Owner

liuch commented Jul 18, 2023

Can you add a setting for it to count all reports or another time period ?

You can change this in the settings, see the status.emails-for-last-n-days option.

I realize that the UI is by no means perfect, but I don't have time or ideas for major tweaks, sorry. Maybe a little later.

@williamdes
Copy link
Contributor Author

I realize that the UI is by no means perfect, but I don't have time or ideas for major tweaks, sorry. Maybe a little later.

okay, what did you think about the Amazon selector UI ?

@liuch
Copy link
Owner

liuch commented Jul 18, 2023

What do you mean? The whole interface you showed in the screenshot? I don't like it. It's too overcomplicated for a simple task. I'm not even sure it can be packaged into a mobile interface. On the other hand, just because I don't like it doesn't mean it's bad. Maybe it's just that my habits are different.

@williamdes
Copy link
Contributor Author

What do you mean? The whole interface you showed in the screenshot? I don't like it. It's too overcomplicated for a simple task. I'm not even sure it can be packaged into a mobile interface. On the other hand, just because I don't like it doesn't mean it's bad. Maybe it's just that my habits are different.

I only mean about the selector where you see 15min... 1d .. custom

@liuch
Copy link
Owner

liuch commented Jul 18, 2023

It's too long. I don't see where it can be placed. How often do you use this option? Is it worth placing on the report list page?

@williamdes
Copy link
Contributor Author

It's too long. I don't see where it can be placed. How often do you use this option? Is it worth placing on the report list page?

You could add the same style of control just above or below the boxes. Not larger than the boxes. It would allow the user to change the filtered time for the statistic

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

No branches or pull requests

2 participants