[PR #160] [MERGED] Add api for mounts #615

Closed
opened 2026-02-05 17:49:13 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/panel/pull/160
Author: @Boy132
Created: 4/26/2024
Status: Merged
Merged: 5/19/2024
Merged by: @iamkubi

Base: mainHead: feature/mount-api


📝 Commits (10+)

  • 46c3e84 add application api endpoints for mounts
  • b071ca4 run pint
  • b44e49b add mounts resource to api key
  • e6edcb9 add includes to mount transformer
  • fcd1ae4 forgot delete route for mount itself
  • 32905cf add migration for "r_mounts" column
  • 4fd40ee add mounts to testcase api key
  • 56e164a Merge branch 'pelican-dev:main' into feature/mount-api
  • 8dc1a96 Merge branch 'pelican-dev:main' into feature/mount-api
  • 7966738 Merge branch 'pelican-dev:main' into feature/mount-api

📊 Changes

12 files changed (+374 additions, -1 deletions)

View changed files

app/Http/Controllers/Api/Application/Mounts/MountController.php (+165 -0)
app/Http/Requests/Api/Application/Mounts/DeleteMountRequest.php (+13 -0)
app/Http/Requests/Api/Application/Mounts/GetMountRequest.php (+13 -0)
app/Http/Requests/Api/Application/Mounts/StoreMountRequest.php (+13 -0)
app/Http/Requests/Api/Application/Mounts/UpdateMountRequest.php (+20 -0)
📝 app/Models/ApiKey.php (+5 -1)
📝 app/Services/Acl/Api/AdminAcl.php (+1 -0)
app/Transformers/Api/Application/MountTransformer.php (+89 -0)
database/migrations/2024_04_28_184102_add_mounts_to_api_keys.php (+28 -0)
📝 lang/en/exceptions.php (+3 -0)
📝 routes/api-application.php (+23 -0)
📝 tests/Integration/Api/Application/ApplicationApiIntegrationTestCase.php (+1 -0)

📄 Description

Closes #130

Adds the following new endpoints to the application api:

Endpoint Description
GET /api/application/mounts list all mounts
GET /api/application/mounts/{mount} view a specific mount
POST /api/application/mounts create a mount
POST /api/application/mounts/{mount}/eggs update the eggs linked to a mount
POST /api/application/mounts/{mount}/nodes update the nodes linked to a mount
PATCH /api/application/mounts/{mount} update a specific mount
DELETE /api/application/mounts/{mount} delete a specific mount
DELETE /api/application/mounts/{mount}/eggs/{egg} remove the link between a mount and an egg
DELETE /api/application/mounts/{mount}/nodes/{node} remove the link between a mount and a node

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/pelican-dev/panel/pull/160 **Author:** [@Boy132](https://github.com/Boy132) **Created:** 4/26/2024 **Status:** ✅ Merged **Merged:** 5/19/2024 **Merged by:** [@iamkubi](https://github.com/iamkubi) **Base:** `main` ← **Head:** `feature/mount-api` --- ### 📝 Commits (10+) - [`46c3e84`](https://github.com/pelican-dev/panel/commit/46c3e84a4fc1cc03ee618f0887bc97694830f191) add application api endpoints for mounts - [`b071ca4`](https://github.com/pelican-dev/panel/commit/b071ca422d9134b4d8a523bb405de30443c95605) run pint - [`b44e49b`](https://github.com/pelican-dev/panel/commit/b44e49b65b66f2cd1fc736faf6f32702899c45b2) add mounts resource to api key - [`e6edcb9`](https://github.com/pelican-dev/panel/commit/e6edcb9361df3d7981d80f00d3b64ed74a32c31b) add includes to mount transformer - [`fcd1ae4`](https://github.com/pelican-dev/panel/commit/fcd1ae4713e70f2718a4b0f0a7a9d3dfb954ba7c) forgot delete route for mount itself - [`32905cf`](https://github.com/pelican-dev/panel/commit/32905cf05b94c2560bcc12274a07a45f8a2f8de6) add migration for "r_mounts" column - [`4fd40ee`](https://github.com/pelican-dev/panel/commit/4fd40ee2c7923ebc8678870879801b050659c43e) add mounts to testcase api key - [`56e164a`](https://github.com/pelican-dev/panel/commit/56e164a2851762b811eb255d236a3e73f6d7c973) Merge branch 'pelican-dev:main' into feature/mount-api - [`8dc1a96`](https://github.com/pelican-dev/panel/commit/8dc1a96be2362318b86bc1b700d20066c14f5ad3) Merge branch 'pelican-dev:main' into feature/mount-api - [`7966738`](https://github.com/pelican-dev/panel/commit/7966738770607d0f112cea9252c9c39b7c4a77a9) Merge branch 'pelican-dev:main' into feature/mount-api ### 📊 Changes **12 files changed** (+374 additions, -1 deletions) <details> <summary>View changed files</summary> ➕ `app/Http/Controllers/Api/Application/Mounts/MountController.php` (+165 -0) ➕ `app/Http/Requests/Api/Application/Mounts/DeleteMountRequest.php` (+13 -0) ➕ `app/Http/Requests/Api/Application/Mounts/GetMountRequest.php` (+13 -0) ➕ `app/Http/Requests/Api/Application/Mounts/StoreMountRequest.php` (+13 -0) ➕ `app/Http/Requests/Api/Application/Mounts/UpdateMountRequest.php` (+20 -0) 📝 `app/Models/ApiKey.php` (+5 -1) 📝 `app/Services/Acl/Api/AdminAcl.php` (+1 -0) ➕ `app/Transformers/Api/Application/MountTransformer.php` (+89 -0) ➕ `database/migrations/2024_04_28_184102_add_mounts_to_api_keys.php` (+28 -0) 📝 `lang/en/exceptions.php` (+3 -0) 📝 `routes/api-application.php` (+23 -0) 📝 `tests/Integration/Api/Application/ApplicationApiIntegrationTestCase.php` (+1 -0) </details> ### 📄 Description Closes #130 Adds the following new endpoints to the application api: | | Endpoint | Description | | - | -| - | | GET | /api/application/mounts | list all mounts | | GET | /api/application/mounts/{mount} | view a specific mount | | POST | /api/application/mounts | create a mount | | POST | /api/application/mounts/{mount}/eggs | update the eggs linked to a mount | | POST | /api/application/mounts/{mount}/nodes | update the nodes linked to a mount | | PATCH | /api/application/mounts/{mount} | update a specific mount | | DELETE | /api/application/mounts/{mount} | delete a specific mount | | DELETE | /api/application/mounts/{mount}/eggs/{egg} | remove the link between a mount and an egg | | DELETE | /api/application/mounts/{mount}/nodes/{node} | remove the link between a mount and a node | --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 17:49:13 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel-pelican-dev#615