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

Use javascript importmaps for Dashboard #574

Merged
merged 1 commit into from
Apr 22, 2022
Merged

Use javascript importmaps for Dashboard #574

merged 1 commit into from
Apr 22, 2022

Conversation

bensheldon
Copy link
Owner

@bensheldon bensheldon commented Apr 19, 2022

This is fairly speculative. I would really like to write modular javascript. The intention is still to avoid dependence on any particular asset pipeline configuration.

  • Constructs import maps manually
  • Adds es-module-shims for non-Chrome compatibility. I tested in Firefox 99 and Safari 15 and it works fine.
  • Refactors Javascript to be more modular. (The polling behavior still needs more work.)
  • Uses vanilla tag.script tags to ensure that script/module paths are only paths and not full urls.; full urls do not work.

@bensheldon bensheldon force-pushed the modules branch 4 times, most recently from 101893c to ee89c21 Compare April 20, 2022 16:04
@bensheldon bensheldon changed the title Set up javascript importmaps for Dashboard Use javascript importmaps for Dashboard Apr 20, 2022
@bensheldon bensheldon force-pushed the modules branch 16 times, most recently from a354f07 to 9e5f1f8 Compare April 20, 2022 21:33
@bensheldon bensheldon marked this pull request as ready for review April 21, 2022 13:49
@bensheldon bensheldon merged commit 95c915e into main Apr 22, 2022
@bensheldon bensheldon deleted the modules branch April 22, 2022 14:03
@bensheldon bensheldon added the refactor Code changes that do not introduce new features label Apr 25, 2022
bkeepers added a commit to bkeepers/good_job that referenced this pull request Apr 29, 2022
* origin/main:
  Release good_job v2.14.1
  Temporarily disable Mass Action "Apply to all" because the action is badly scoped (bensheldon#583)
  Release good_job v2.14.0
  Add mass update operations for jobs to Dashboard (bensheldon#578)
  Track down incompatibility/race condition between JRuby and RSpec mocks in tests (bensheldon#581)
  Release good_job v2.13.2
  Namespaces assets per Rails docs (bensheldon#580)
  Release good_job v2.13.1
  Set up javascript importmaps for Dashboard; refactor Polling (bensheldon#574)
  Use toasts to show notices and alerts (bensheldon#577)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Code changes that do not introduce new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant