[Portal] Add time picker to Audit Log custom date range dialog - #5831
Draft
jimmycwc wants to merge 6 commits into
Draft
[Portal] Add time picker to Audit Log custom date range dialog#5831jimmycwc wants to merge 6 commits into
jimmycwc wants to merge 6 commits into
Conversation
Match Fraud Protection Logs by enabling DateRangeDialog time picker, storing datetime in the URL, and showing the selected range label in the filter dropdown. Co-authored-by: Cursor <cursoragent@cursor.com>
Use end-of-day for rangeTo defaults and legacy yyyy-MM-dd bookmarks so exclusive backend bounds keep the selected day inclusive. Co-authored-by: Cursor <cursoragent@cursor.com>
Align the time select with Authgear datetime display instead of 24-hour clock. Co-authored-by: Cursor <cursoragent@cursor.com>
Update audit log and fraud protection dialog copy for datetime pickers, and give Analytics its own date-only labels. Co-authored-by: Cursor <cursoragent@cursor.com>
Keep filter labels consistent with DateTimePicker display format. Co-authored-by: Cursor <cursoragent@cursor.com>
Copy the start picker's time onto the end date when no time is set yet, round clamped times down to a dropdown increment, and format AM/PM manually so Chinese locales do not show 上午/下午. Co-authored-by: Cursor <cursoragent@cursor.com>
Author
|
@tung2744 Added a time picker to Audit Log and verified the changes using the review-pr. |
jimmycwc
marked this pull request as draft
July 27, 2026 08:16
jimmycwc
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
DateRangeDialog+DateTimePickerwith time picking enabled, storing full ISO datetime in URL instead ofyyyy-MM-ddDateTimePickertime display and parsing from 24-hour to 12-hour AM/PM, matching Authgear's existing datetime display conventionsyyyy-MM-ddtobookmarks now parse as end-of-day (inclusive), and selecting an end date for the first time defaults to 23:59:59 instead of 00:00:00customRangeLabeltoAuditLogFilterBarso the selected date/time range is shown in the filter buttonen-USinformatCustomDateRangeLabelso filter labels match the AM/PM display in the pickerTest plan
from=2026-07-01&to=2026-07-21— verify it is interpreted as Jul 1 00:00 AM → Jul 21 11:59 PM (whole days inclusive)Made with Cursor