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

[FEATURE] NPM package #37

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

[FEATURE] NPM package #37

wants to merge 6 commits into from

Conversation

wlaeri
Copy link
Contributor

@wlaeri wlaeri commented Mar 5, 2020

This PR adds a prepack script to the package.json that compiles the viking.js output along with necessary dependencies so that we can publish the package to npm.

@wlaeri wlaeri requested review from waratuman, malomalo and bemky March 5, 2020 20:50
@wlaeri
Copy link
Contributor Author

wlaeri commented Mar 5, 2020

As a side note, I think we can remove Backbone and JQuery as strict dependencies. Should shave off about 1.5MB from the package size.

@bemky
Copy link
Collaborator

bemky commented Apr 9, 2021

With release of v2, viking has moved to a full esm module, and is now installable via npm package.

@bemky bemky closed this Apr 9, 2021
malomalo added 2 commits April 9, 2021 16:10
# By Ben Ehmke (6) and Jon Bracy (1)
# Via GitHub
* master:
  Association Order (#52)
  move load and loaded events on association to only fire when actual load (#51)
  Change View.$ to querySelector instead of querySelectorAll (#49)
  Application waits for itself to render before displaying a view (#48)
  Automatically add X-CSRF-TOKEN header if csrf-token available in the document (#46)
  add return for map, and forEach alias (#47)
  Viking  V2 (#44)

# Conflicts:
#	Gemfile.lock
#	package-lock.json
#	package.json
#	viking.js
@malomalo
Copy link
Owner

malomalo commented Apr 9, 2021

Hmmm I do want some of the stuff here tho

@malomalo malomalo reopened this Apr 9, 2021
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