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

Remove deprecated helper PageUrl #3747

Closed
wants to merge 1 commit into from
Closed

Conversation

rohan013
Copy link

Fixes #2217

-Removed function
-Removed associated tests

@ErisDS ErisDS added this to the Future milestone Aug 13, 2014
@ErisDS
Copy link
Member

ErisDS commented Aug 13, 2014

Looking around at themes on GitHub, it looks like very few theme developers have gotten the message that this has been deprecated. I think I need to do some more work to get this message out there before merging this.

@ErisDS
Copy link
Member

ErisDS commented Sep 2, 2014

In the interest of housekeeping, I'm going to close this PR, when we're ready for it, I can reopen and merge it.

@ErisDS ErisDS closed this Sep 2, 2014
@ErisDS ErisDS modified the milestone: Future Backlog Oct 13, 2015
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.

Remove the deprecated helper pageUrl
2 participants