Skip to content

Conversation

@Agrendalath
Copy link
Member

This adds a version constraint for LTI Consumer XBlock before we merge #31475.

@Agrendalath Agrendalath requested a review from robrap January 30, 2023 14:02
@openedx-webhooks openedx-webhooks added the blended PR is managed through 2U's blended developmnt program label Jan 30, 2023
@openedx-webhooks
Copy link

Thanks for the pull request, @Agrendalath!

When this pull request is ready, tag your edX technical lead.

Copy link
Contributor

@robrap robrap left a comment

Choose a reason for hiding this comment

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

In the future, I’d use <7.3 so you don’t block bug fixes. Shouldn’t matter here. Thanks.

@robrap robrap merged commit 426c23b into openedx:master Jan 30, 2023
@openedx-webhooks
Copy link

@Agrendalath 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.

@Agrendalath Agrendalath deleted the agrendalath/add_lti_consumer_xblock_version_constraint branch January 30, 2023 14:18
@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

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

Labels

blended PR is managed through 2U's blended developmnt program

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants