Skip to content

Conversation

@matiasperrone-exo
Copy link
Contributor

@matiasperrone-exo matiasperrone-exo added the documentation Improvements or additions to documentation label Oct 24, 2025
@matiasperrone-exo matiasperrone-exo self-assigned this Oct 24, 2025
@matiasperrone-exo matiasperrone-exo added the review Need reviewing from the developer label Nov 10, 2025
@matiasperrone-exo matiasperrone-exo removed the review Need reviewing from the developer label Nov 24, 2025
@smarcet smarcet force-pushed the main branch 4 times, most recently from c6ecdd0 to 728ae67 Compare December 17, 2025 00:43
Copy link

@caseylocker caseylocker left a comment

Choose a reason for hiding this comment

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

  1. Resolve merge conflicts - PR is currently CONFLICTING
  2. Fix path - Change /api/v1/files/upload → /api/public/v1/files/upload
  3. Fix security scheme - Replace [['bearer' => []]] with a defined OAuth2 scheme (e.g., summit_oauth2) or define a new security scheme
  4. Fix tag casing - Change files → Files

Should this be moved from UploadController::upload() to OAuth2ChunkedFilesApiController::uploadFile() since that's what the route uses in public_api.php?
Route::post('upload', 'OAuth2ChunkedFilesApiController@uploadFile');

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants