-
Notifications
You must be signed in to change notification settings - Fork 509
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
fix(iis): merge web.config
#1658
Conversation
β Live Preview ready!
|
web.config
Sorry it took long to check on this @Hardaker587. I will make local review maybe adding changes to make XML lib optional. (you don't need to rebase on each commit) |
Thanks so much - just been making sure my fork is up to date with the latest :) |
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.
Thanks. Made a couple of refactors to make XML dependency optional (you have to install it in your project). I hope you are happy with the changes otherwise feel free to make an amended PR.
Perfect, I will make a PR with amended docs to reflect this |
π Linked issue
Resolves #1628
β Type of change
π Description
@pi0 Adding additional functionality to #1436 in order to prevent breaking changes for those already integrated with IIS
Note: I reapplied the changes as there was my web.config in the original commits
Additional notes:
π Checklist