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

feat(match2): support postponed #5174

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Conversation

hjpalpha
Copy link
Collaborator

@hjpalpha hjpalpha commented Dec 5, 2024

Summary

support postponed as a new status in matches

How did you test this change?

dev

@hjpalpha hjpalpha changed the title idea for martin feat(match2): support postponed Dec 5, 2024
@hjpalpha
Copy link
Collaborator Author

hjpalpha commented Dec 5, 2024

needs testing @iMarbot :P

@hjpalpha hjpalpha requested review from Rathoz and iMarbot December 16, 2024 15:07
@hjpalpha hjpalpha marked this pull request as ready for review December 16, 2024 15:07
Copy link
Collaborator

@iMarbot iMarbot left a comment

Choose a reason for hiding this comment

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

In MatchGroupInputUtil.getMatchStatus, np it treated with priority, but in MatchGroupInputUtil.matchIsFinished, postponed is treated with priority. Should we stick to one "priority" order?

@hjpalpha hjpalpha requested a review from iMarbot January 6, 2025 07:02
@hjpalpha
Copy link
Collaborator Author

nay blockers for this? @Rathoz

@Rathoz
Copy link
Collaborator

Rathoz commented Jan 14, 2025

DateIsExact should probably be set to false, other fields may need updates too. Not sure about using finished=true in countdown

@iMarbot
Copy link
Collaborator

iMarbot commented Jan 15, 2025

Will dateextact being false hide the previous time? imo, we should still show the old time for postponed matches.

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

Successfully merging this pull request may close these issues.

3 participants