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.

Sign up for GitHub

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

Jump to bottom

remove flask upper pin #1246

Closed
wants to merge 2 commits into from
Closed

remove flask upper pin #1246

wants to merge 2 commits into from

Conversation

utnapischtim
Copy link
Contributor

  • global: make forward compatible to sqlalchemy >= 2
  • fix: for compatibility with sqlalchemy >= 2

@utnapischtim
global: make forward compatible to sqlalchemy >= 2
27bb0a6
@utnapischtim
fix: for compatibility with sqlalchemy >= 2
19eb31d 
* the problem is on test_members_services.py. if test_add_denied runs it
  creates a situation where the curator role with number 3 is converted to
  string. which is the reason that the needs can't find an intersection
  because the need in the identity has still an int 3.
@utnapischtim utnapischtim force-pushed the remove-flask-upper-pin branch from 2b6f2d3 to 19eb31d Compare November 15, 2024 21:59
@utnapischtim
Copy link
Contributor Author

seems not necessary for flask>=3.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@utnapischtim