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

Replace Query with Path #11320

Merged
merged 1 commit into from
Feb 9, 2021
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions lib/galaxy/webapps/galaxy/api/configuration.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@

from fastapi import (
Depends,
Query,
Path,
)
from fastapi_utils.cbv import cbv
from fastapi_utils.inferring_router import InferringRouter as APIRouter
Expand Down Expand Up @@ -53,7 +53,7 @@

AdminUserRequired = Depends(get_admin_user)

EncodedIdQueryParam: EncodedDatabaseIdField = Query(
EncodedIdPathParam: EncodedDatabaseIdField = Path(
...,
title='Encoded id',
description='Encoded id to be decoded',
Expand Down Expand Up @@ -125,7 +125,7 @@ def dynamic_tool_confs(self) -> List[Dict[str, str]]:
def decode_id(
self,
trans: ProvidesAppContext = Depends(get_trans),
encoded_id: EncodedDatabaseIdField = EncodedIdQueryParam
encoded_id: EncodedDatabaseIdField = EncodedIdPathParam
) -> Dict[str, int]:
"""Decode a given id."""
return self.configuration_manager.decode_id(encoded_id)
Expand Down