Skip to content

Commit

Permalink
Merge pull request #40257 from nextcloud/fix/openapi/comments/ignore
Browse files Browse the repository at this point in the history
comments: Ignore endpoints in OpenAPI
  • Loading branch information
provokateurin authored Sep 5, 2023
2 parents f331d86 + 8f0a35e commit eb1b09d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 53 deletions.
2 changes: 2 additions & 0 deletions apps/comments/lib/Controller/NotificationsController.php
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@
namespace OCA\Comments\Controller;

use OCP\AppFramework\Controller;
use OCP\AppFramework\Http\Attribute\IgnoreOpenAPI;
use OCP\AppFramework\Http\NotFoundResponse;
use OCP\AppFramework\Http\RedirectResponse;
use OCP\AppFramework\Http;
Expand All @@ -40,6 +41,7 @@
/**
* @package OCA\Comments\Controller
*/
#[IgnoreOpenAPI]
class NotificationsController extends Controller {

protected IRootFolder $rootFolder;
Expand Down
54 changes: 1 addition & 53 deletions apps/comments/openapi.json
Original file line number Diff line number Diff line change
Expand Up @@ -41,58 +41,6 @@
}
}
},
"paths": {
"/index.php/apps/comments/notifications/view/{id}": {
"get": {
"operationId": "notifications-view",
"summary": "View a notification",
"tags": [
"notifications"
],
"security": [
{},
{
"bearer_auth": []
},
{
"basic_auth": []
}
],
"parameters": [
{
"name": "id",
"in": "path",
"description": "ID of the notification",
"required": true,
"schema": {
"type": "string"
}
}
],
"responses": {
"303": {
"description": "Redirected to notification",
"headers": {
"Location": {
"schema": {
"type": "string"
}
}
}
},
"404": {
"description": "Notification not found",
"content": {
"text/html": {
"schema": {
"type": "string"
}
}
}
}
}
}
}
},
"paths": {},
"tags": []
}

0 comments on commit eb1b09d

Please sign in to comment.