scottdurow / SparkleXrm

An open-source library for building Dynamics CRM XRM solutions using Script#, jQuery & Knockoutjs.
MIT License
267 stars 197 forks source link

Webresource dependencies support #187

Open lukasbelak opened 6 years ago

lukasbelak commented 6 years ago

Currently we can update spkl.json file with something like:

{ "webresources": [ { "files": [{ "uniquename": "new_/js/account.js", "file": "js\account.js", "description": "desc" } ] }}

D365 supports webresource dependencies from v9 now, so we do not need to add required webresources to each form separately. webdep

It would be great if we could extend config above to something like:

{ "webresources": [ { "files": [{ "uniquename": "new/js/account.js", "file": "js\account.js", "description": "desc", "dependencies": [{ "uniquename": "new/js/common.js", }] } ] }}

Please, give some thought to it.

scottdurow commented 6 years ago

totally agree - I was thinking along the same lines. I also think it would be great to allow form event registrations as well as part of the webresources config section.

lukasbelak commented 6 years ago

hi @scottdurow ..please, wait with this enhancement, I have already started to work on this, deploy-webresource part is almost done, the remaining bit is download-webresources-config. Hopefully, I will send you my solution to review this week.

EDIT: hi @scottdurow , I experience some troubles when working on download-webresources task. When I forked your solution, I see 'get-webresources' task in RunTask function, but no bat file for executing it. Then when I updated nuget package to latest 1.0.196-beta version. New bat file called download-webresources.bat showed up and there is trigger for 'download-webresources' task, not 'get-webresources' task I see after forking the solution. Is it true that you did not commit the latest changes, because the latest release is a beta version? Will you commit your latest changes or should I send you just deploy-webresources solution with webresource dependencies enhancement?

EDIT: ok, I just noticed, there is a v9 branch, gonna do the changes there

parvezghumra commented 4 years ago

Hi @scottdurow @lukasbelak

Do you have any updates on or planned delivery timeline for this web resource dependency issue?

Thanks, Parvez