Skip to content

Implement milestone anomalies warnings#2

Open
SAN-MUYUN wants to merge 13 commits intomainfrom
feature/milestone-anomalies-warnings
Open

Implement milestone anomalies warnings#2
SAN-MUYUN wants to merge 13 commits intomainfrom
feature/milestone-anomalies-warnings

Conversation

@SAN-MUYUN
Copy link
Copy Markdown
Owner

@SAN-MUYUN SAN-MUYUN commented Jul 17, 2025

Summary:

Type of change:

  • ✨ New Feature/ Enhancement
  • 🧪 Tests Update

Changes Made:

Implemented a new model MilestoneAnomaly to represent a milestone anomaly.

When milestone has one of the following anomalies, a warning will be shown above the issues and PRs section. The warning will display both the milestone title and its anomaly.

  • Milestone with no deadline
  • Open milestone that has gone past deadline
  • Closed milestone with open issues or unmerged PR
  • Multiple active milestones

Screenshots:

Before:

No warning is shown for milestone anomalies:

image

After:

When any milestones has anomalies:
image

When there are too many milestone anomalies, it will be kept at a fixed height. The component will become scrollable
Width of this component will follow the width of the issues/PRs section

image

Proposed Commit Message:

Implement warnings to display basic milestone anomalies for milestones

There is no warning if a milestone does not have a deadline or has
gone past deadline.

Let's display a warning above the milestone filter if a milestone
without a deadline or has gone past deadline is selected.

Checklist:

  • I have tested my changes thoroughly.
  • I have created tests for any new code files created in this PR or provided a link to a issue/PR that addresses this.
  • I have added or modified code comments to improve code readability where necessary.
  • I have updated the project's documentation as necessary.

@damithc
Copy link
Copy Markdown

damithc commented Jul 19, 2025

@SAN-MUYUN Good work.
In the next iteration, you can experiment with other ways of displaying these notifications.

e.g., consolidating all of them into one message and showing it at the top of issue/PR side.

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.

2 participants