Skip to content
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

Revert "refactor: changed survey report message location and added a new info…" #34110

Merged
merged 1 commit into from
Jan 24, 2024

Conversation

macdiesel
Copy link
Contributor

@macdiesel macdiesel commented Jan 24, 2024

Reverts #34033

This PR is currently preventing enterprise from managing learners within enterprises as jquery is now loaded twice on django admin pages.

This is the offending line from the PR:

<script type="text/javascript" src="{% static 'common/js/vendor/jquery.js' %}"></script>

Jira: https://2u-internal.atlassian.net/browse/ENT-8310

@ormsbee @Asespinel

@macdiesel macdiesel enabled auto-merge January 24, 2024 22:29
@macdiesel
Copy link
Contributor Author

@felipemontoya FYI since you were the approver on the initial PR.

@macdiesel macdiesel merged commit bca34c0 into master Jan 24, 2024
63 checks passed
@macdiesel macdiesel deleted the revert-34033-ase/refactor-survey-report branch January 24, 2024 22:48
@edx-pipeline-bot
Copy link
Contributor

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

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

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