feat: show plugin update time in marketplace details - #9690
Merged
Conversation
Soulter
marked this pull request as ready for review
August 14, 2026 17:06
Contributor
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- The
updatedAtDisplaycomputed value relies onnew Date(value), which can behave inconsistently for non-ISO or timezone-less strings; consider normalizing the timestamp format or using a dedicated date parsing/formatting utility to ensure consistent results across browsers. - If the project has a centralized date/time formatting helper, it might be preferable to reuse it instead of creating a new
Intl.DateTimeFormathere so that date formats are consistent across the dashboard and easier to adjust in one place.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The `updatedAtDisplay` computed value relies on `new Date(value)`, which can behave inconsistently for non-ISO or timezone-less strings; consider normalizing the timestamp format or using a dedicated date parsing/formatting utility to ensure consistent results across browsers.
- If the project has a centralized date/time formatting helper, it might be preferable to reuse it instead of creating a new `Intl.DateTimeFormat` here so that date formats are consistent across the dashboard and easier to adjust in one place.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
BegoniaHe
pushed a commit
to Xero-Team/AstrBot
that referenced
this pull request
Aug 15, 2026
Upstream-Commit: 70b95e0 Upstream-Author: Soulter <37870767+Soulter@users.noreply.github.com> Upstream-PR: AstrBotDevs#9690 Sync-Disposition: adapt Fork-Adaptation: Keep the fork's Chinese and English locale set while formatting marketplace timestamps with the current i18n locale. Tested: cd dashboard && pnpm test
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
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
updated_atvalue in the plugin detail information tableWhy
Marketplace entries already include an update timestamp, but the plugin detail page did not expose it. Displaying it helps users understand how recently a plugin was maintained before installing it.
Validation
pnpm buildpnpm typecheckuv run ruff format --check .uv run ruff check .Summary by Sourcery
Display marketplace plugin update time in the plugin detail view using localized formatting.
New Features:
Enhancements: