Skip to content

Conversation

@bbearce
Copy link
Collaborator

@bbearce bbearce commented Nov 14, 2022

@ mention of reviewers

@Didayolo
@Tthomas63

A brief description of the purpose of the changes contained in this PR.

Upon manual UI challenge creation, the leaderboard isn't saved.

Issues this PR resolves

733 (#733)

Known issues to be addressed in a separate PR

N/A

A checklist for hand testing

  • Make a challenge using the form in the UI and don't forget to create a leaderboard.
  • Next once competition is created, go to edit and make sure the leaderboard you created still exists.
  • Also go to results tab and make sure you can see the leaderboard.

Any relevant files for testing

N/A

Misc. comments

I'm worried that this fix and the underlying code I copied or updating a challenge is not what the original creators wanted. See this comment above the code I copied:

# TODO - This is Temporary. Need to change Leaderboard to Phase connect to M2M and handle this correctly.
.

Checklist

  • Code review by me
  • Hand tested by me
  • I'm proud of my work
  • Code review by reviewer
  • Hand tested by reviewer
  • Ready to merge

@Didayolo
Copy link
Member

@bbearce good job
This solves half the issue if I understand correctly?

@bbearce
Copy link
Collaborator Author

bbearce commented Nov 27, 2022

Well at first yes but really there are two issues. (1) is the leaderboard which is fixed and the other (2) was the queue. I broke those into two separate issues with the first being this one and the second here #740.

@Didayolo Didayolo closed this Jun 1, 2023
@Didayolo Didayolo deleted the manual_ui_challenge_creation_issue_733 branch June 1, 2023 15:01
@bbearce bbearce mentioned this pull request Jun 6, 2023
7 tasks
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