Skip to content

[BACKPORT] fix: Hide enterprise/login button if enterprise integration disabled. - #27834

Merged
pdpinch merged 1 commit into
openedx:open-release/lilac.masterfrom
raccoongang:fix/BTR-52
Jun 7, 2021
Merged

[BACKPORT] fix: Hide enterprise/login button if enterprise integration disabled.#27834
pdpinch merged 1 commit into
openedx:open-release/lilac.masterfrom
raccoongang:fix/BTR-52

Conversation

@cmltaWt0

@cmltaWt0 cmltaWt0 commented Jun 4, 2021

Copy link
Copy Markdown
Contributor

Backport for https://github.com/edx/edx-platform/pull/27130 to Lilac.master

Description:

Fix an issue with an existing button for enterprise login on the login page
when enterprise integration is disabled.

Jira
BTR issue

…openedx#27130)

* fix: Hide enterprise/login button if enterprise integration disabled.

[BTR-52](https://openedx.atlassian.net/browse/BTR-52)
Fix an issue with existing button for enterprise login on the login page
when enterprise integration is disabled.

* Address review comments
@openedx-webhooks openedx-webhooks added needs triage open-source-contribution PR author is not from Axim or 2U labels Jun 4, 2021
@openedx-webhooks

Copy link
Copy Markdown

Thanks for the pull request, @cmltaWt0! I've created OSPR-5835 to keep track of it in JIRA, where we prioritize reviews. Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket:

  • supporting documentation
  • Open edX discussion forum threads
  • timeline information ("this must be merged by XX date", and why that is)
  • partner information ("this is a course on edx.org")
  • any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

@natabene

natabene commented Jun 4, 2021

Copy link
Copy Markdown
Contributor

@cmltaWt0 Thank you for your contribution. FYI @nedbat

@pdpinch
pdpinch merged commit 335ab58 into openedx:open-release/lilac.master Jun 7, 2021
@openedx-webhooks

Copy link
Copy Markdown

@cmltaWt0 🎉 Your pull request was merged!

Please take a moment to answer a two question survey so we can improve your experience in the future.

dmitry-viskov pushed a commit to weaveeducation/edx-platform that referenced this pull request Jun 8, 2021
…openedx#27130) (openedx#27834)

* fix: Hide enterprise/login button if enterprise integration disabled.

[BTR-52](https://openedx.atlassian.net/browse/BTR-52)
Fix an issue with existing button for enterprise login on the login page
when enterprise integration is disabled.

Co-authored-by: Igor Degtiarov <igor.degtiarov@raccoongang.com>
@GlugovGrGlib
GlugovGrGlib deleted the fix/BTR-52 branch September 20, 2023 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged open-source-contribution PR author is not from Axim or 2U

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants