Comments (8)
Looking at nvQuickTheme their gulp build tasks also manage the information in the manifest file
from generator-dnn.
After some discussions, we are going to change the packageInstall and packageSource to pull the version from the manifest file. Then we are going to add a manifest task that will update the version of the manifest.
from generator-dnn.
I do believe the version of the Dlls is set through the package.json configuration via a gulp task that updates the Assembly Info. So if the manifest is not being updated that is a bug. The initial idea was that gulp would drive the build process. Eventually, I would like the dnn manifest file to be generated for the user so they do not have to manage it. For example, the program would store the user's information in the package.json file and then during build a process would look for any folder with a View file in it and consider that a module and add it to the dnn.manifest file and then package the installer. I also believe that is the way nvQuickTheme is generating the manifest file (Uses config values from the package.json and populates the dnn.manifest on build)
from generator-dnn.
I have some problems with the package.json managing the manifest file.
- What if a user needs to modify the manifest file for their project and the gulp tasks keep overwriting it?
- What if a user needs to make their manifest file include multiple modules. One extension that installs multiple dnn modules
- What if the user needs to add a non-default home route in case of MVC
There is going to be a fine line between making things easier and having gulp tasks do it all for us and taking away control from people where the tool might not be as useful.
With all that being said I see a lot of merit in having gulp manage the dnn manifest file. I personally find it cumbersome in how my modules are managed from a build, versioning and manifest file perspective.
This is going to require more thought.
from generator-dnn.
It will be hard to strike a balance. There may have to be some gulp tasks that are run optionally. If someone uses the npm version command then there should be a hook that would update the manifest versions after the version has been patched.
Or maybe we give instructions to disable a build step if the user wants to manually manage their manifest file. That might be a really good option considering this tool was meant to attract some newer developers.
from generator-dnn.
That is correct @mtrutledge - we also have a separate JSON file for making project setup a bit easier. That way people do not have to understand all the intricacies of the DNN manifest file.
from generator-dnn.
My question is where does this leave the power users? The people that are comfortable modifying the manifest file? They need to find portions of the gulp tasks and update them to not overwrite their changes
from generator-dnn.
This is a great compromise where we can still take advantage of all the gulp scripts but still take into account the power users that want to manage the manifest themselves.
Do you want help with this one? We should just need an xml reader that we include and grab the file version. The only issue that may come up is with old manifest files the version isn't set the same way, but I don't think it is a problem we will run into with this tooling. As a safeguard we could follow this workflow:
- Check the dnn manifest file first
- Check the node package.json file 2nd
this way we have a fallback in case the manifest file is messed up for whatever reason
from generator-dnn.
Related Issues (20)
- Create an Angular option for SPA modules HOT 7
- PDB generated by command line msbuild is not allowing Visual Studio to Debug HOT 4
- Alert the user that a new version of the generator is out using the update-notifier HOT 1
- Ability to add modules to an existing solution HOT 3
- Intellisense errors in views of newly generated module HOT 2
- Build ReactJS SPA Template with gulp HOT 1
- Update from Yeoman-Generator 2.0.5 to 3.1.1 HOT 2
- Upgrade from Gulp 3 to 4 HOT 1
- GHPages/Docs Link on main README.md
- Add GitHub Topics to repo HOT 1
- Update repo Description and README to use optimum keywords HOT 1
- Gulp Scripts Included in Release Build HOT 5
- DNNModuleAuthorize access set to edit by default on template HOT 1
- Upgrade NPM Packages
- RFC: Thoughts on moving to yarn HOT 13
- Sln will not build on Mac Visual Studio. Windows won't Build Client from SLN build HOT 6
- Update docs
- Could not parse JSON in file HOT 1
- Whata does this do? HOT 3
- SPA Module does not generate install.zip file on Install buil HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from generator-dnn.