html5Mode support.

[rhutchison]

Open for discussion. This pull request enables use of html5Mode. This should also still work with html5Mode(false).

[uwolfer]

I'm using this patch set and it seems to work without issues. Thanks for sharing.

[rhutchison]

This has been updated to leverage public/config.js. I changed the default to false, as I am told this is the more common implementation. I will be adding it to generator with a param as part of the application setup.

[sunshineo]

This is very helpful!
I vote for have the default set to true. There are a lot of benefit with copy paste link and SEO using html5Mode true.
I even vote for not give the configuration option and default to html5Mode true. Convention over configuration. MEAN.JS is all about pick the convention for the user. If user want the flexibility, use just Node+Express and they can plug and play.

---

With some more readings, I feel I may be wrong about what I said above.

[tylerdmace]

I vote we leave 'false' as default. I don't believe SEO nor pretty URLs are going to be of importance to most applications.

[ilanbiala]

@tylerdmace most care about pretty URLs and SEO, because that's how you grow. Besides it doesn't add much trouble, just that you add 1 route that sends the index.html template instead of a 404, and you pass back the original url and let Angular do all the routing other than API.

[tylerdmace]

There's no way to easily gauge the implementation type of most MEAN applications so I digress. However, I have changed my mind on the issue since you're right in that having the support doesn't really have any drawbacks. The downsides for not having it are poor SEO support and ugly URLs but the downside for having it are... well, nothing, really - but we need to have a way to fall back to hashbang URLs for people who need the older tech for support reasons.

[ilanbiala]

that's what Angular does anyway. You just do $locationProvider.html5Mode(true).hashPrefix('!'); like @rhutchison did in the PR.

[amoshaviv]

added in the new version

[ebraminio]

Where it is merged? I can not see it neither on 0.4 branch nor master.

[xxorax]

I confirm, the routes were not merged.

[woniesong92]

Is it merged?

[simison]

@xxorax @woniesong92 @ebraminio perhaps here? https://github.com/meanjs/mean/tree/0.4.0

[monad98]

@rhutchison
I still have problem. When I refresh the page at "/articles", I am automatically signed out.
The other pages like "/articles/create", "/settings/profile" is ok.

EDIT: This problem also happens without html5mode. :(

[rhutchison]

@monad98 this PR is pretty out dated from upstream, so not sure how reliable it would be. I know @amoshaviv manually plucked these changes and put them in to the 0.4.0 branch (ab81d61#diff-90593c1684669b4a0cc185ec424f0a24R9) - have you had any luck there?