Skip to content

Removed lines causing issues with iis 10 #1646

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

Merged
merged 1 commit into from
Sep 5, 2017
Merged

Conversation

hitautodestruct
Copy link
Contributor

Removed the lines causing issues in windows server 2016 iis version 10.
Added more exact instructions for adding a config file.

Removed the lines causing issues in windows server 2016 iis version 10.
Added more exact instructions for adding a config file.
Copy link
Member

@posva posva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I cannot verify the information because I don't use IIS but it looks good
Thanks!

@hitautodestruct
Copy link
Contributor Author

@posva when will this get merged into the public docs?

@posva
Copy link
Member

posva commented Aug 9, 2017

I can merge it now but it won't be deployed yet anyway
edit: sorry for the close 😓

@posva posva closed this Aug 9, 2017
@posva posva reopened this Aug 9, 2017
@hitautodestruct
Copy link
Contributor Author

hitautodestruct commented Sep 5, 2017

Hey @posva

Will this ever be merged?

@LinusBorg
Copy link
Member

@hitautodestruct We will, yes.

@posva We shouldn't have to wait with deployment imho, or is there a specific reason to?

@posva
Copy link
Member

posva commented Sep 5, 2017

@LinusBorg Since last accident (quite some months ago) I haven't reconfigured deployment of vue-router docs, but it's something I have to do it very soon. Let's merge this and the other accepted docs PR and deploy. If you have everything setup for the deploy, ping me on slack so I know I don't have to deploy 🙂

@posva posva merged commit 2393f65 into vuejs:dev Sep 5, 2017
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