Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: compatibility with werkzeug >= 3.0.0. #311

Merged

Conversation

utnapischtim
Copy link
Contributor

  • to keep the functionality as it is, a method from werkzeug have been
    copy pasted over and changed slightly.

* to keep the functionality as it is, a method from werkzeug have been
  copy pasted over and changed slightly.
* this change is a working solution for sqlalchemy ~= 1.4 but a
  necessity for >= 2.0
* LegacyAPIWarning: The Query.get() method is considered legacy as of
  the 1.x series of SQLAlchemy and becomes a legacy construct in 2.0.
  The method is now available as Session.get() (deprecated since:
  2.0) (Background on SQLAlchemy 2.0 at: https://sqlalche.me/e/b8d9)
  role = current_datastore.role_model.query.get(id_)
@utnapischtim utnapischtim force-pushed the remove-flask-upper-pin branch from 0f06ce7 to 3bd1228 Compare November 5, 2024 09:03
@utnapischtim utnapischtim merged commit a9a74fe into inveniosoftware:master Nov 5, 2024
2 checks passed
@utnapischtim utnapischtim deleted the remove-flask-upper-pin branch November 5, 2024 09:13
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant