Skip to content

MCKIN-22514 round off correctly for v3 - #297

Merged
aayushagrawal9 merged 1 commit into
open-craft:problem-builder-v3from
mudassir-hafeez:MCKIN-22514-v3
Aug 25, 2020
Merged

MCKIN-22514 round off correctly for v3#297
aayushagrawal9 merged 1 commit into
open-craft:problem-builder-v3from
mudassir-hafeez:MCKIN-22514-v3

Conversation

@mudassir-hafeez

Copy link
Copy Markdown
Contributor

Jira ticket https://edx-wiki.atlassian.net/browse/MCKIN-22514

There is an endpoint that was missed in #267 which needs to be addressed.

@mudassir-hafeez

Copy link
Copy Markdown
Contributor Author

@xitij2000 here is the PR for v3. Can you please review it?

@aayushagrawal9 aayushagrawal9 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good to go

@aayushagrawal9
aayushagrawal9 merged commit 687bc77 into open-craft:problem-builder-v3 Aug 25, 2020
@mudassir-hafeez
mudassir-hafeez deleted the MCKIN-22514-v3 branch August 25, 2020 17:13
@mudassir-hafeez

Copy link
Copy Markdown
Contributor Author

Thank you @aayushagra for reviewing this PR. we have the same #296 for the master branch. Can you please review that too?

@aayushagrawal9

Copy link
Copy Markdown
Contributor

👍

  • I tested this: Tested that changes work and all tests pass
  • I read through the code
  • I checked for accessibility issues
  • Includes documentation
  • I made sure any change in configuration variables is reflected in the corresponding client's configuration-secure repository.

@mudassir-hafeez

Copy link
Copy Markdown
Contributor Author

@aayushagra It seems like you didn't create the release tag version as it should be. I think it would be a typo and should be v3.4.23 prefix semantic version with “v” rather than this 3.4.23, and same for 4.0.7

@xitij2000

Copy link
Copy Markdown
Member

@mudassir-hafeez I've updated the tags. We've been a bit inconsistent about this at times.

@mudassir-hafeez

Copy link
Copy Markdown
Contributor Author

@xitij2000 thank you for updating the tags.

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