Skip to content

Commit 5b47633

Browse files
fix: change in public endpoints, removal of unnecessary scope security
1 parent 0c8450c commit 5b47633

2 files changed

Lines changed: 30 additions & 34 deletions

File tree

app/Http/Controllers/Apis/Protected/Summit/OAuth2SummitSponsorApiController.php

Lines changed: 29 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -298,27 +298,10 @@ public function getAllBySummitV2($summit_id){
298298

299299
#[OA\Get(
300300
path: "/api/public/v1/summits/all/{id}/sponsors",
301-
description: "Get a specific sponsor by id",
301+
description: "Get all sponsors for a summit",
302302
summary: 'Get Sponsors by Summit ID (Public)',
303303
operationId: 'getSponsorsV2BySummitPublic',
304304
tags: ['Sponsors (Public)'],
305-
security: [
306-
[
307-
'summit_sponsor_oauth2' => [
308-
SummitScopes::ReadSummitData,
309-
SummitScopes::ReadAllSummitData,
310-
]
311-
]
312-
],
313-
x: [
314-
'required-groups' => [
315-
IGroup::SuperAdmins,
316-
IGroup::Administrators,
317-
IGroup::SummitAdministrators,
318-
IGroup::Sponsors,
319-
IGroup::SponsorExternalUsers,
320-
]
321-
],
322305
parameters: [
323306
new OA\Parameter(
324307
name: 'id',
@@ -327,22 +310,13 @@ public function getAllBySummitV2($summit_id){
327310
schema: new OA\Schema(type: 'integer'),
328311
description: 'The summit id'
329312
),
330-
new OA\Parameter(
331-
name: 'sponsor_id',
332-
in: 'path',
333-
required: true,
334-
schema: new OA\Schema(type: 'integer'),
335-
description: 'The sponsor id'
336-
),
337313
],
338314
responses: [
339315
new OA\Response(
340316
response: Response::HTTP_OK,
341317
description: 'Success',
342318
content: new OA\JsonContent(ref: '#/components/schemas/SponsorV2')
343319
),
344-
new OA\Response(response: Response::HTTP_UNAUTHORIZED, description: "Unauthorized"),
345-
new OA\Response(response: Response::HTTP_FORBIDDEN, description: "Forbidden"),
346320
new OA\Response(response: Response::HTTP_NOT_FOUND, description: "Not Found"),
347321
new OA\Response(response: Response::HTTP_INTERNAL_SERVER_ERROR, description: "Server Error")
348322
]
@@ -359,10 +333,27 @@ public function getAllBySummitPublic($summit_id)
359333

360334
#[OA\Get(
361335
path: "/api/v2/summits/{id}/sponsors/{sponsor_id}",
362-
description: "Get all sponsors for a summit (Public API)",
363-
summary: 'Read Sponsors by Summit',
336+
description: "Get a specific sponsor by id",
337+
summary: 'Read Sponsor by Summit',
364338
operationId: 'getSponsorsV2BySummit',
365339
tags: ['Sponsors'],
340+
security: [
341+
[
342+
'summit_sponsor_oauth2' => [
343+
SummitScopes::ReadSummitData,
344+
SummitScopes::ReadAllSummitData,
345+
]
346+
]
347+
],
348+
x: [
349+
'required-groups' => [
350+
IGroup::SuperAdmins,
351+
IGroup::Administrators,
352+
IGroup::SummitAdministrators,
353+
IGroup::Sponsors,
354+
IGroup::SponsorExternalUsers,
355+
]
356+
],
366357
parameters: [
367358
new OA\Parameter(
368359
name: 'id',
@@ -371,6 +362,13 @@ public function getAllBySummitPublic($summit_id)
371362
schema: new OA\Schema(type: 'integer'),
372363
description: 'The summit id'
373364
),
365+
new OA\Parameter(
366+
name: 'sponsor_id',
367+
in: 'path',
368+
required: true,
369+
schema: new OA\Schema(type: 'integer'),
370+
description: 'The sponsor id'
371+
),
374372
],
375373
responses: [
376374
new OA\Response(
@@ -2057,7 +2055,7 @@ public function getAd($summit_id, $sponsor_id, $ad_id)
20572055

20582056
#[OA\Get(
20592057
path: "/api/v1/summits/{id}/sponsors/{sponsor_id}/materials",
2060-
description: "required-groups " . IGroup::SuperAdmins . ", " . IGroup::Administrators . ", " . IGroup::SummitAdministrators . ", " . IGroup::Sponsors . ", " . IGroup::SponsorExternalUsers,
2058+
description: "required-groups " . IGroup::SuperAdmins . ", " . IGroup::Administrators . ", " . IGroup::SummitAdministrators . ", " . IGroup::Sponsors,
20612059
summary: 'Read Sponsor Materials',
20622060
operationId: 'getSponsorMaterials',
20632061
tags: ['Sponsors'],
@@ -2067,7 +2065,6 @@ public function getAd($summit_id, $sponsor_id, $ad_id)
20672065
IGroup::Administrators,
20682066
IGroup::SummitAdministrators,
20692067
IGroup::Sponsors,
2070-
IGroup::SponsorExternalUsers,
20712068
]
20722069
],
20732070
security: [
@@ -2512,7 +2509,7 @@ public function getMaterial($summit_id, $sponsor_id, $material_id)
25122509

25132510
#[OA\Get(
25142511
path: "/api/v1/summits/{id}/sponsors/{sponsor_id}/social-networks",
2515-
description: "required-groups " . IGroup::SuperAdmins . ", " . IGroup::Administrators . ", " . IGroup::SummitAdministrators . ", " . IGroup::Sponsors . ", " . IGroup::SponsorExternalUsers,
2512+
description: "required-groups " . IGroup::SuperAdmins . ", " . IGroup::Administrators . ", " . IGroup::SummitAdministrators . ", " . IGroup::Sponsors,
25162513
summary: 'Read Sponsor Social Networks',
25172514
operationId: 'getSponsorSocialNetworks',
25182515
tags: ['Sponsors'],
@@ -2522,7 +2519,6 @@ public function getMaterial($summit_id, $sponsor_id, $material_id)
25222519
IGroup::Administrators,
25232520
IGroup::SummitAdministrators,
25242521
IGroup::Sponsors,
2525-
IGroup::SponsorExternalUsers,
25262522
]
25272523
],
25282524
security: [

app/Swagger/Models/SubQuestionRuleSchema.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
new OA\Property(property: 'last_edited', type: 'integer', example: 1, format: "time_epoch"),
1515
new OA\Property(property: 'visibility', type: 'string'),
1616
new OA\Property(property: 'visibility_condition', type: 'string'),
17-
new OA\Property(property: 'answer_values', type: 'string_array'),
17+
new OA\Property(property: 'answer_values', type: 'array', items: new OA\Items(type: 'string')),
1818
new OA\Property(property: 'answer_values_operator', type: 'string'),
1919
new OA\Property(property: 'order', type: 'integer'),
2020
new OA\Property(property: 'sub_question_id', type: 'integer'),

0 commit comments

Comments
 (0)