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

Marathon Match created in Direct shows up in the Active matches, even if the Status is Draft #462

Open
RiteshTC opened this issue Oct 10, 2018 · 4 comments
Labels
P2 P2 - High Priority

Comments

@RiteshTC
Copy link

A Marathon Match created in Direct. The status is set to Draft.
https://www.topcoder.com/direct/projectOverview.action?formData.projectId=18893
The match start date is set in past.

The match is shown in active marathon matches in the Contest Page
https://community.topcoder.com/longcontest/?m%20odule=ViewProblemStatement

The contest date has been temporarily set in future to avoid the issue for now.

@ajefts @skyhit

@ajefts
Copy link
Contributor

ajefts commented Oct 10, 2018

cc: @cwdcwd

@skyhit Do we need to set the round status and/or round_segment dates in a better way?

@skyhit
Copy link
Collaborator

skyhit commented Oct 11, 2018

@RiteshTC @ajefts @cwdcwd
is the match created with past start date, and a future end date?

@RiteshTC
Copy link
Author

It was created with future start and end date. It was created sometime in Sep. Oct 4th was set the Match Start Date

@skyhit
Copy link
Collaborator

skyhit commented Oct 17, 2018

@RiteshTC @ajefts @cwdcwd
so there is the situation, for active mms, it is just based on the start date and end date, see https://github.com/topcoder-platform/tc-database-scripts/blob/dev/informixoltp/query_tool/query_long_contest_active_contests.txt

probably, we should do

  1. When creating from direct app, it should set a non-active status
  2. improve the query to only include active status

@RiteshTC RiteshTC added the P1 P1 - Highest Priority Issue label Feb 15, 2019
@RiteshTC RiteshTC added P2 P2 - High Priority and removed P1 P1 - Highest Priority Issue labels Feb 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 P2 - High Priority
Projects
None yet
Development

No branches or pull requests

3 participants