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

Fix errors with raising FullResultSet exception and with alter_column_type_sql() and collate_sql() functions #229

Merged
merged 5 commits into from Feb 21, 2023

Conversation

khanhmaibui
Copy link
Contributor

@khanhmaibui khanhmaibui commented Feb 16, 2023

This PR fixes several errors caused by upgrading to Django 4.2.

Changes include:

  • Properly handling FullResultSet exception in certain cases.
  • alter_column_type_sql() function now takes 7 arguments instead of 5.
  • collate_sql() function now takes 4 arguments instead of 2.

@khanhmaibui khanhmaibui marked this pull request as draft February 16, 2023 23:24
@khanhmaibui khanhmaibui reopened this Feb 16, 2023
@khanhmaibui khanhmaibui changed the title Fix error with raising FullResultSet exception Fix errors with raising FullResultSet exception and with alter_column_type_sql() and collate_sql() functions Feb 17, 2023
@khanhmaibui khanhmaibui marked this pull request as ready for review February 17, 2023 21:42
@khanhmaibui khanhmaibui merged commit 81018de into django42 Feb 21, 2023
@khanhmaibui khanhmaibui deleted the raise-full-result-set-fix branch February 21, 2023 20:22
mShan0 added a commit that referenced this pull request May 31, 2023
* Allow Django 4.2 (#227)

* Allow Django 4.2

* allow Django 4.2

* Fix errors with raising FullResultSet exception and with alter_column_type_sql() and collate_sql() functions (#229)

* fix error with raising fullresultset

* add django4.2 condition

* fix alter_column_type_sql and collate_sql to take 2 additional arguments

* delete argument 'old_rel_collation'

* fix arguments names

* fix last_executed_query() to properly replace placeholders with params (#234)

* disable allows_group_by_select_index

* unskip old tests

* unskip some tests

* skip more tests

* Use latest Django 4.2 beta for tox (#238)

* use 4.2 rc1 branch (#240)

* allow partial support for filtering against window functions (#239)

* add subsecond support to Now() (#242)

* assign value to display_size (#244)

* add latest django 4.2 branch to ci

* raise an error when batch_size is zero. (#259)

* replicate get or create test for mssql (#265)

* Add skipped tests to Django 4.2 (#268)

* skip django 4.2 failing tests

* skip schema test

* skip aggregate annotation pruning test

---------

Co-authored-by: mShan0 <mark.shan19@gmail.com>

* syntax fix

* ci fix

* bump version to 1.3

---------

Co-authored-by: Khanh Bui <85855766+khanhmaibui@users.noreply.github.com>
Co-authored-by: Khanh Bui <khanhmb815@gmail.com>
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.

None yet

2 participants