-
-
Notifications
You must be signed in to change notification settings - Fork 244
[deps] Upgrade Django 5.x and channels dependencies from openwisp-utils extras #1001
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
Conversation
| path = reverse('geo_api:detail_location', args=[l1.pk]) | ||
| data = {'type': 'outdoor'} | ||
| with self.assertNumQueries(8): | ||
| with self.assertNumQueries(9): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The number of queries increased because of these changes openwisp/django-loci#157
- Dropped support for Python < 3.9 - Dropped support for Django < 4.2 Closes #987
Django 5.2 requires Postgresql 14+
| template_qs = Template.objects.filter(type='vpn') | ||
| self.assertEqual(template_qs.count(), 1) | ||
| t = template_qs.first() | ||
| self.assertDictContainsSubset(_original_context, t.get_context()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is removed in Python 3.12
| self.assertIsNone(c1.device.management_ip) | ||
| self.assertEqual(c2.device.management_ip, '192.168.1.99') | ||
| # other organization is not affected | ||
| self.assertEquals(c3.device.last_ip, '127.0.0.1') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is removed in Python 3.12
nemesifier
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍
| return context | ||
|
|
||
| with context: | ||
| func(*args, **kwargs) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, thanks!
nemesifier
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Checklist
Blockers