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

Set module's assetpath attribute to fix component's importPath() #83

Closed
wants to merge 6 commits into from

Conversation

mjcctech
Copy link

When component is loaded by polymer-webpack-loader it lose information about original path. Polymer provide importPatch property(https://www.polymer-project.org/2.0/docs/devguide/dom-template#urls-in-templates). But to work it correctly with webpack loader assetpath attribute should be set on module.

@jsf-clabot
Copy link

jsf-clabot commented Apr 18, 2018

CLA assistant check
All committers have signed the CLA.

@codecov
Copy link

codecov bot commented Apr 18, 2018

Codecov Report

Merging #83 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #83      +/-   ##
==========================================
+ Coverage   97.02%   97.04%   +0.02%     
==========================================
  Files           2        2              
  Lines         269      271       +2     
  Branches       65       67       +2     
==========================================
+ Hits          261      263       +2     
  Misses          8        8
Impacted Files Coverage Δ
src/index.js 97.4% <100%> (+0.01%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cf8fa98...58c87c6. Read the comment docs.

mjcctech added 5 commits April 18, 2018 14:21
Set module's assetpath attribute to fix component's importPath()
It is possible to specify directory where components are installed. Default value is './src'
@ChadKillingsworth
Copy link
Member

Sorry for the delay - I somehow missed this PR.

Can you rebase your changes onto master and remove the references to your local version?

@mjcctech
Copy link
Author

mjcctech commented Aug 5, 2018

I didn't manage to clean up on this branch so I created new merge request #88

@mjcctech mjcctech closed this Aug 5, 2018
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