Skip to content

Provide the loader with created elements#292

Merged
ocombe merged 3 commits intoocombe:masterfrom
msak:master
Oct 13, 2016
Merged

Provide the loader with created elements#292
ocombe merged 3 commits intoocombe:masterfrom
msak:master

Conversation

@msak
Copy link
Contributor

@msak msak commented Mar 11, 2016

Your loader decorators are very straight forward and can be overridden for custom purpose.
But I miss the possibility to get access to the elements created by buildElement.

It would be great if you could accept this request with my little change.

@ocombe
Copy link
Owner

ocombe commented Mar 11, 2016

Sure, could you change it in the sources instead of the dist folder please ?

@msak
Copy link
Contributor Author

msak commented Mar 11, 2016

Shame on me :)
Do you able to take the second commit from this pull request, or should I create a new one?

@ocombe
Copy link
Owner

ocombe commented Mar 11, 2016

If you add a commit to the same branch, it will be added to the PR (which is what you did).

@msak
Copy link
Contributor Author

msak commented Mar 11, 2016

So already done!

If you merge, when will be a new npm package created?

@ocombe
Copy link
Owner

ocombe commented Mar 11, 2016

I'll try to make a new release this week end.

@msak
Copy link
Contributor Author

msak commented Mar 11, 2016

Great! Thanks!

@msak
Copy link
Contributor Author

msak commented Mar 22, 2016

Hi Olivier, could you create a new npm package?

@kination
Copy link
Collaborator

kination commented Sep 5, 2016

Hello @msak . If it's okay, could you remove the changes on /dist files?

@ocombe
Copy link
Owner

ocombe commented Sep 5, 2016

they will be overwritten when you rebuild them anyway @djKooks

@kination
Copy link
Collaborator

kination commented Sep 5, 2016

Oh, okay. Just want to make sure.

@kination
Copy link
Collaborator

@ocombe It looks fine to merge this. How do you think?

@ocombe ocombe merged commit 3351e44 into ocombe:master Oct 13, 2016
@ocombe
Copy link
Owner

ocombe commented Oct 13, 2016

Yes :)

@ocombe
Copy link
Owner

ocombe commented Feb 3, 2017

I just released 1.1.0 with this, I think I did it right, I haven't published a new version on this repo for ages, let me know if it works

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