Skip to content
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

Add deprecated warning to readme #24

Merged
merged 1 commit into from
Jan 21, 2017
Merged

Add deprecated warning to readme #24

merged 1 commit into from
Jan 21, 2017

Conversation

simison
Copy link
Member

@simison simison commented Jan 16, 2017

Google has deprecated (10/2015) _escaped_fragment_ URL parameter and can now scrape javascript generated pages and thus this package isn't needed anymore at MEAN.js projects.

Read more:

@simison
Copy link
Member Author

simison commented Jan 16, 2017

Ping @roieki & @lirantal — I recon after merging this the repo could be abandoned and people would stop wasting time opening new issues here and at the main MEAN repo.

@lirantal
Copy link
Member

I never maintained this repo so I'm not sure I can speak for @roieki but I guess he hasn't be that active and if it causes confusion then we can probably close it.

Maybe we can pull in @codydiag or @ilanbiala to confirm/raise concerns?

@lirantal lirantal self-assigned this Jan 16, 2017
@simison
Copy link
Member Author

simison commented Jan 16, 2017

@lirantal this could be cleaned out as well: https://meanjs.org/modules.html

@lirantal
Copy link
Member

yeah that makes sense, thanks.

@ilanbiala
Copy link
Member

I've never actually worked with this repo either. I think a deprecated/caution header is good if Google has fixed the issue.

@lirantal lirantal merged commit 534c3da into meanjs:master Jan 21, 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