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

Do not use deprecated methods #1737

Merged
merged 2 commits into from
Feb 25, 2020

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Feb 25, 2020

What is this pull request for?

We should not use deprecated methods internally.
Otherwise they will log deprecations that users cannot do anything about.

This method has been deprecated and since we do not want to log
deprecation warnings we must not use it.
This method has been deprecated and we do not want to use it
internally to avoid loggin deprecation warnings.
@tvdeyen tvdeyen force-pushed the do-not-dogfeed-deprecated-methods branch from 80656d3 to ad31bc3 Compare February 25, 2020 11:34
@tvdeyen tvdeyen merged commit 96b8f62 into AlchemyCMS:master Feb 25, 2020
@tvdeyen tvdeyen deleted the do-not-dogfeed-deprecated-methods branch February 25, 2020 11:43
tvdeyen added a commit that referenced this pull request Feb 25, 2020
- Do not use deprecated methods [#1737](#1737) ([tvdeyen](https://github.com/tvdeyen))
- Order contents by their position in its element [#1733](#1733) ([tvdeyen](https://github.com/tvdeyen))
- Eager load relations in elements trash [#1732](#1732) ([tvdeyen](https://github.com/tvdeyen))
- Run CI builds with Sprockets 3.7.2 [#1731](#1731) ([tvdeyen](https://github.com/tvdeyen))
- Re-organize development dependencies [#1730](#1730) ([tvdeyen](https://github.com/tvdeyen))
- Update pr template [#1729](#1729) ([tvdeyen](https://github.com/tvdeyen))
- Generate views without _view in the filename [#1728](#1728) ([tvdeyen](https://github.com/tvdeyen))
- Fix CI Builds [#1727](#1727) ([tvdeyen](https://github.com/tvdeyen))
- Fix page tagging condition: should_attach_to_menu?  [#1725](#1725) ([mickenorlen](https://github.com/mickenorlen))
- Fix Alchemy.user_class_name constant conflict [#1724](#1724) ([mickenorlen](https://github.com/mickenorlen))
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.

1 participant