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

22 add lti tool api #24

Merged
merged 14 commits into from
Aug 20, 2024
Merged

22 add lti tool api #24

merged 14 commits into from
Aug 20, 2024

Commits on Aug 20, 2024

  1. refactor: Add LtiToolView, urls, serialziers

    @geoff-va
    geoff-va committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    bf40c4b View commit details
    Browse the repository at this point in the history

  2. test: Test create tool

    @geoff-va
    geoff-va committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    a3fb518 View commit details
    Browse the repository at this point in the history

  3. refactor: Restrict keys to those within specd org

    @geoff-va
    geoff-va committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    3dd39e3 View commit details
    Browse the repository at this point in the history

  4. test: key dne, org dne, key in other org

    @geoff-va
    geoff-va committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    d5d384d View commit details
    Browse the repository at this point in the history

  5. refactor: Allow serializer to handle unique constraint 

    - super().create call already handles integrity error
    @geoff-va
    geoff-va committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    48a3d73 View commit details
    Browse the repository at this point in the history

  6. test: issuer/client_id already exists returns 400

    @geoff-va
    geoff-va committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    2d82ab1 View commit details
    Browse the repository at this point in the history

  7. fix: LtiToolKey -> LtiTool in LtiToolViewSet

    @geoff-va
    geoff-va committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    1244e31 View commit details
    Browse the repository at this point in the history

  8. test: list returns tools for specified org only

    @geoff-va
    geoff-va committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    c2b853b View commit details
    Browse the repository at this point in the history

  9. test: delete tool/key not in specified org fails

    @geoff-va
    geoff-va committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    9731f5a View commit details
    Browse the repository at this point in the history

  10. fix: autoimported ... is invalid for serializer data init 

    - data param was defualting to ... which is technically passing
something in so it was throwing unexpected errors
- Use *args, **kwargs to preserve original defaults
    @geoff-va
    geoff-va committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    b77cdba View commit details
    Browse the repository at this point in the history

  11. test: Finish testing LtiTool endpoint

    @geoff-va
    geoff-va committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    880d037 View commit details
    Browse the repository at this point in the history

  12. build: Bump version to 2.3.0

    @geoff-va
    geoff-va committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    835cc7d View commit details
    Browse the repository at this point in the history

  13. docs: Add 2.3.0 to ChangeLog

    @geoff-va
    geoff-va committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    3c63991 View commit details
    Browse the repository at this point in the history

  14. chore: rm unused functions

    @geoff-va
    geoff-va committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    81a80a2 View commit details
    Browse the repository at this point in the history