Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Commit

Permalink
fix: revert view changes
Browse files Browse the repository at this point in the history
  • Loading branch information
irtazaakram committed Aug 29, 2023
1 parent ee4361c commit a50d340
Show file tree
Hide file tree
Showing 10 changed files with 13 additions and 29 deletions.
3 changes: 2 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,12 +13,13 @@ jobs:
name: Tests
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
matrix:
os:
- ubuntu-20.04
python-version:
- 3.8
toxenv: [ django32 ]
toxenv: [ django32, django42 ]
steps:
- uses: actions/checkout@v1
- name: setup python
Expand Down
20 changes: 1 addition & 19 deletions registrar/apps/api/v3/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
"""
import logging

from edx_api_doc_tools import query_parameter, schema_for
from edx_rest_framework_extensions.auth.jwt.authentication import (
JwtAuthentication,
)
Expand All @@ -20,24 +19,7 @@

logger = logging.getLogger(__name__)

SCHEMA_COMMON_RESPONSES = {
401: 'User is not authenticated.',
405: 'HTTP method not supported on this path.'
}


@schema_for(
'get',
parameters=[
query_parameter('org_key', str, 'Organization filter'),
query_parameter('user_has_perm', str, 'Permission filter'),
],
responses={
403: 'User lacks access to organization.',
404: 'Organization does not exist.',
**SCHEMA_COMMON_RESPONSES,
},
)

class ProgramListPaginationView(ProgramListView, TrackViewMixin, ListAPIView):
"""
A view for listing program objects.
Expand Down
2 changes: 1 addition & 1 deletion requirements/base.txt
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ django-simple-history==3.0.0
# via
# -c requirements/common_constraints.txt
# -r requirements/base.in
django-storages==1.8
django-storages==1.10.1
# via
# -c requirements/constraints.txt
# -r requirements/base.in
Expand Down
2 changes: 1 addition & 1 deletion requirements/constraints.txt
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ Django<4.0
django-extensions<3.2.0

# django-storages version 1.9 drops support for boto storage backend.
django-storages<1.9
django-storages<1.10.1

# From base.in
celery<5.0 # version 5.0 drops support for python 3.5
Expand Down
2 changes: 1 addition & 1 deletion requirements/devstack.txt
Original file line number Diff line number Diff line change
Expand Up @@ -173,7 +173,7 @@ django-simple-history==3.0.0
# via
# -c requirements/common_constraints.txt
# -r requirements/local.txt
django-storages==1.8
django-storages==1.10.1
# via
# -c requirements/constraints.txt
# -r requirements/local.txt
Expand Down
2 changes: 1 addition & 1 deletion requirements/local.txt
Original file line number Diff line number Diff line change
Expand Up @@ -173,7 +173,7 @@ django-simple-history==3.0.0
# via
# -c requirements/common_constraints.txt
# -r requirements/test.txt
django-storages==1.8
django-storages==1.10.1
# via
# -c requirements/constraints.txt
# -r requirements/test.txt
Expand Down
2 changes: 1 addition & 1 deletion requirements/monitoring/requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -264,7 +264,7 @@ django-simple-history==3.0.0
# -r requirements/monitoring/../local.txt
# -r requirements/monitoring/../production.txt
# -r requirements/monitoring/../test.txt
django-storages==1.8
django-storages==1.10.1
# via
# -r requirements/monitoring/../devstack.txt
# -r requirements/monitoring/../local.txt
Expand Down
2 changes: 1 addition & 1 deletion requirements/production.txt
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ django-simple-history==3.0.0
# via
# -c requirements/common_constraints.txt
# -r requirements/base.txt
django-storages==1.8
django-storages==1.10.1
# via
# -c requirements/constraints.txt
# -r requirements/base.txt
Expand Down
2 changes: 1 addition & 1 deletion requirements/test.txt
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,7 @@ django-simple-history==3.0.0
# via
# -c requirements/common_constraints.txt
# -r requirements/base.txt
django-storages==1.8
django-storages==1.10.1
# via
# -c requirements/constraints.txt
# -r requirements/base.txt
Expand Down
5 changes: 3 additions & 2 deletions tox.ini
Original file line number Diff line number Diff line change
@@ -1,13 +1,14 @@
[tox]
envlist = py38-django{32}
envlist = py38-django{32, 42}
skipsdist = true

[testenv]
passenv = *
deps =
django32: -r requirements/django.txt
django42: Django>=4.2,<4.3
-r{toxinidir}/requirements/test.txt
whitelist_externals =
i18n_tool
/bin/bash
commands = {posargs:pytest}
commands = {posargs:pytest}

0 comments on commit a50d340

Please sign in to comment.