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

Fix how global dependencies are loaded in the UI #244

Merged
merged 2 commits into from Sep 22, 2017

Conversation

gconsidine
Copy link
Contributor

SUMMARY

The UI has a handful of dependencies attached to window (angular,
jquery, lodash, etc). In the case of schedules, jquery was included
and extended as expected, but then clobbered by another module. This
prevented the Select2 dependency from working as expected.

Rather than relying on webpack to export particular dependencies as
global, that work is done in the vendor entry point now instead.

ISSUE TYPE
COMPONENT NAME
  • UI

The UI has a handful of dependencies attached to window (angular,
jquery, lodash, etc). In the case of schedules, jquery was included
and extended as expected, but then clobbered by another module. This
prevented the Select2 dependency from working as expected.

Rather than relying on webpack to export particular dependencies as
global, that work is done in the vendor entry point now instead.
Copy link
Contributor

@jlmitch5 jlmitch5 left a comment

Choose a reason for hiding this comment

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

+1!

@gconsidine gconsidine merged commit 3c062b3 into ansible:devel Sep 22, 2017
@gconsidine gconsidine deleted the ui/fix-schedules branch September 22, 2017 20:36
@naisanza naisanza mentioned this pull request Sep 23, 2017
@seffyroff
Copy link

seffyroff commented Feb 17, 2018

I'm seeing the same issues described above after deploying from the install guide with docker-compose - did the PR above make it into master, or should I sync up to devel?

EDIT: looks like devel==master around here. I guess the merge didn't fix the issue i'm experiencing.

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

Successfully merging this pull request may close these issues.

None yet

4 participants