Skip to content

Conversation

kmkkiii
Copy link
Collaborator

@kmkkiii kmkkiii commented Mar 26, 2025

サマリ

概要、背景

#9301

(不具合の場合のみ) 発生原因

find_dataset_object, delete_dataset_objectのobjectName引数に/が含まれていると、URLとして認識されてしまい404エラーになってしまっていました。

対応内容

やったこと

  • find_dataset_object, delete_dataset_objectでAPIリクエスト前にobjectNameをエンコードするように修正しました。
  • endpoint変更
  • exampleの修正

やれていないこと、妥協点

UI/UX

before

after

テスト

変更の意図に沿った基本動作が確認できている

  • find_dataset_objectメソッドが実行できること
  • delete_dataset_objectメソッドが実行できること

関連する既存機能にデグレがないことを確認

エッジケースや例外パターンの動作を確認

関連リンク

補足

@kmkkiii kmkkiii self-assigned this Mar 26, 2025
@kmkkiii kmkkiii requested a review from ryoKaz March 26, 2025 06:48
Copy link
Contributor

@ryoKaz ryoKaz left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@ryoKaz ryoKaz left a comment

Choose a reason for hiding this comment

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

LGTM

@ryoKaz ryoKaz merged commit 1fed2a2 into main Mar 27, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants