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

Reset Filters button on /ratelimits/:id/logs redirects to /audit page #2394

Closed
3 tasks done
unrenamed opened this issue Oct 13, 2024 · 23 comments · Fixed by #2703
Closed
3 tasks done

Reset Filters button on /ratelimits/:id/logs redirects to /audit page #2394

unrenamed opened this issue Oct 13, 2024 · 23 comments · Fixed by #2703
Assignees
Labels

Comments

@unrenamed
Copy link
Contributor

Preliminary Checks

Reproduction / Replay Link (Optional)

No response

Issue Summary

The Reset Filters button on the /ratelimits/:id/logs page incorrectly redirects users to the /audit page instead of resetting the filters on the current logs page.

Steps to Reproduce

  1. Navigate to the /ratelimits/:id/logs page.
  2. Apply any filters to the logs (e.g., identifier, IP address) that result in no data being returned.
  3. Click the Reset Filters button.
  4. Observe the redirection to the /audit page instead of the expected behavior.

Expected behavior

When the Reset Filters button is clicked, the user should remain on the /ratelimits/:id/logs page. All applied filters should be cleared, displaying the default log view without redirection to another page.

Other information

No response

Screenshots

image

Version info

- OS:
- Node:
- npm:
@unrenamed unrenamed added Bug Something isn't working Needs Approval Needs approval from Unkey labels Oct 13, 2024
Copy link

linear bot commented Oct 13, 2024

@perkinsjr perkinsjr added 🕹️ oss.gg 🕹️ 150 points and removed Needs Approval Needs approval from Unkey labels Oct 13, 2024
@perkinsjr
Copy link
Collaborator

/award 150

Copy link

oss-gg bot commented Oct 13, 2024

Awarding unrenamed: 150 points 🕹️ Well done! Check out your new contribution on oss.gg/unrenamed

@chetansharma-meta
Copy link

/assign

Copy link

oss-gg bot commented Oct 13, 2024

Assigned to @chetansharma-meta! Please open a draft PR linking this issue within 48h ⚠️ If we can't detect a PR from you linking this issue in 48h, you'll be unassigned automatically 🕹️ Excited to have you ship this 🚀

@shashankpatil28
Copy link

/assign

Copy link

oss-gg bot commented Oct 13, 2024

This issue is already assigned to another person. Please find more issues here.

@Sans25VS
Copy link

/assign

Copy link

oss-gg bot commented Oct 14, 2024

This issue is already assigned to another person. Please find more issues here.

@saiteja-in
Copy link

/assign

Copy link

oss-gg bot commented Oct 14, 2024

This issue is already assigned to another person. Please find more issues here.

@spawade99
Copy link

/assign

Copy link

oss-gg bot commented Oct 15, 2024

This issue is already assigned to another person. Please find more issues here.

@div000-321go
Copy link

/assign

Copy link

oss-gg bot commented Oct 15, 2024

This issue is already assigned to another person. Please find more issues here.

Copy link

oss-gg bot commented Oct 18, 2024

@chetansharma-meta, Just a little reminder: Please open a draft PR linking this issue within 12 hours. If we can't detect a PR in 12h, you will be unassigned automatically.

@mohdshubair313
Copy link

/assign

Copy link

oss-gg bot commented Oct 18, 2024

This issue is already assigned to another person. Please find more issues here.

@mohdshubair313
Copy link

/assign

Copy link

oss-gg bot commented Oct 19, 2024

This issue is already assigned to another person. Please find more issues here.

1 similar comment
Copy link

oss-gg bot commented Oct 19, 2024

This issue is already assigned to another person. Please find more issues here.

@Vardhaman619
Copy link
Contributor

/assign

Copy link

oss-gg bot commented Oct 21, 2024

This issue is already assigned to another person. Please find more issues here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

10 participants