Skip to content

Conversation

@pandafy
Copy link
Member

@pandafy pandafy commented Apr 3, 2025

@pandafy pandafy moved this from To do (general) to Needs review in OpenWISP Contributor's Board Apr 3, 2025
path = reverse('geo_api:detail_location', args=[l1.pk])
data = {'type': 'outdoor'}
with self.assertNumQueries(8):
with self.assertNumQueries(9):
Copy link
Member Author

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
@pandafy pandafy moved this from Needs review to In progress in OpenWISP Contributor's Board Apr 4, 2025
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())
Copy link
Member Author

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')
Copy link
Member Author

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

@pandafy pandafy changed the title [deps] Install channels dependencies from openwisp-utils extras [deps] Upgrade Django 5.x and channels dependencies from openwisp-utils extras Apr 4, 2025
@coveralls
Copy link

coveralls commented Apr 4, 2025

Coverage Status

coverage: 98.884%. remained the same
when pulling d6f7fc2 on channels-deps
into 4881326 on master.

Copy link
Member

@nemesifier nemesifier left a 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)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks!

Copy link
Member

@nemesifier nemesifier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@nemesifier nemesifier merged commit e00abf6 into master Apr 4, 2025
30 of 31 checks passed
@nemesifier nemesifier deleted the channels-deps branch April 4, 2025 22:03
@github-project-automation github-project-automation bot moved this from In progress to Done in OpenWISP Contributor's Board Apr 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

4 participants