Skip to content

Add legacyReviewId to reviews #144

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

Merged
merged 10 commits into from
Dec 4, 2019

Conversation

meshde
Copy link
Contributor

@meshde meshde commented Sep 21, 2019

  • Add legacyReviewId to review endpoints
  • Add review.legacyReviewId filter to GET /submissions endpoint

Fix for #143

@callmekatootie
Copy link
Contributor

@cwdcwd This one's good to merge

Copy link
Contributor

@cwdcwd cwdcwd left a comment

Choose a reason for hiding this comment

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

Please change from a UUID string to a number field

@meshde
Copy link
Contributor Author

meshde commented Sep 24, 2019

Please change from a UUID string to a number field

@cwdcwd updated

Dushyant Bhalgami and others added 6 commits September 24, 2019 16:12
…_146

Fix lint errors and force pre commit lint checks
…re/crate_review_queued_status

review create with option score for queued status
…re/crate_review_queued_status

updated createReview schema for null score for queued submission
@callmekatootie
Copy link
Contributor

@cwdcwd The merge conflicts are resolved.

@cwdcwd cwdcwd merged commit c374b03 into topcoder-platform:develop Dec 4, 2019
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