-
Notifications
You must be signed in to change notification settings - Fork 751
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
Removed Module Creator from distribution #4208
Conversation
- Removes the Module Creator admin module from the distribution as per it's move to https://github.com/DNNCommunity/Dnn.ModuleCreator/releases/tag/v9.8.1 - Also reduces build warnings by about 46
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there an issue discussing the removal of this module? I don't recall discussing this one.
Oh sorry, forgot to link the isssue #3620 Basically the reasoning was that it's something for developers and should not be part of the base install. On the linked repository I took the existing code and fixed all warnings for deprecations and made it a standalone buildable solution. People who do need this module can grab it from there and it makes the base distribution lighter. |
Are we good with this in 9.8.1 or we slate it for later (10) ? Just for context, it would not affect upgrades but new installs would not have the ModuleCreator in it. |
@valadas I am thinking it is good for 9.8.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome! Thanks @valadas 🎉
I put a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm good with 9.8.1 for this
As of DNN 9.8.1, Module Creator is not included in the distribution of DNN (see dnnsoftware#4208)
As of DNN 9.8.1, Module Creator is not included in the distribution of DNN (see dnnsoftware#4208)
Closes #3620