Skip to content

Show year in notice list dates - #17

Open
pelagiann wants to merge 2 commits into
Rewritefrom
notice-date-year
Open

Show year in notice list dates#17
pelagiann wants to merge 2 commits into
Rewritefrom
notice-date-year

Conversation

@pelagiann

@pelagiann pelagiann commented May 20, 2026

Copy link
Copy Markdown

Updated notice list date formatting to always include the year.
Screenshot 2026-05-20 145516

Date filter summary also shows the year.
image

Update notice list date formatting to always include the year.
Copilot AI review requested due to automatic review settings May 20, 2026 09:30

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates notice list date rendering so each notice row always shows a year, improving clarity for older notices.

Changes:

  • Simplified NoticeCell date formatting to always include the year (MMM Do, YYYY).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/components/notice/NoticeCell.js
@rtb-12

rtb-12 commented May 20, 2026

Copy link
Copy Markdown

Final Review — Safe to merge ✅

Trivial 4-line change, low risk.

  • Copilot's earlier comment about the filter summary missing the year is already addressed in the current diff (NoticeList.js now formats both endpoints as MMM Do, YYYY).
  • Cosmetic only: 'to''-' separator in the filter summary. Confirm with design that this is intentional.

Verdict: Ship it.

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.

3 participants