Skip to content

Commit 1f25062

Browse files
committed
API: Updated docs with consistent types, fixed users response example
For #5178 and #5183
1 parent 9f68ca5 commit 1f25062

File tree

3 files changed

+11
-12
lines changed

3 files changed

+11
-12
lines changed

app/Uploads/Controllers/AttachmentApiController.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -171,16 +171,16 @@ protected function rules(): array
171171
{
172172
return [
173173
'create' => [
174-
'name' => ['required', 'min:1', 'max:255', 'string'],
174+
'name' => ['required', 'string', 'min:1', 'max:255'],
175175
'uploaded_to' => ['required', 'integer', 'exists:pages,id'],
176176
'file' => array_merge(['required_without:link'], $this->attachmentService->getFileValidationRules()),
177-
'link' => ['required_without:file', 'min:1', 'max:2000', 'safe_url'],
177+
'link' => ['required_without:file', 'string', 'min:1', 'max:2000', 'safe_url'],
178178
],
179179
'update' => [
180-
'name' => ['min:1', 'max:255', 'string'],
180+
'name' => ['string', 'min:1', 'max:255'],
181181
'uploaded_to' => ['integer', 'exists:pages,id'],
182182
'file' => $this->attachmentService->getFileValidationRules(),
183-
'link' => ['min:1', 'max:2000', 'safe_url'],
183+
'link' => ['string', 'min:1', 'max:2000', 'safe_url'],
184184
],
185185
];
186186
}

app/Users/Controllers/UserApiController.php

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -37,27 +37,28 @@ protected function rules(int $userId = null): array
3737
{
3838
return [
3939
'create' => [
40-
'name' => ['required', 'min:2', 'max:100'],
40+
'name' => ['required', 'string', 'min:2', 'max:100'],
4141
'email' => [
42-
'required', 'min:2', 'email', new Unique('users', 'email'),
42+
'required', 'string', 'email', 'min:2', new Unique('users', 'email'),
4343
],
4444
'external_auth_id' => ['string'],
4545
'language' => ['string', 'max:15', 'alpha_dash'],
46-
'password' => [Password::default()],
46+
'password' => ['string', Password::default()],
4747
'roles' => ['array'],
4848
'roles.*' => ['integer'],
4949
'send_invite' => ['boolean'],
5050
],
5151
'update' => [
52-
'name' => ['min:2', 'max:100'],
52+
'name' => ['string', 'min:2', 'max:100'],
5353
'email' => [
54-
'min:2',
54+
'string',
5555
'email',
56+
'min:2',
5657
(new Unique('users', 'email'))->ignore($userId ?? null),
5758
],
5859
'external_auth_id' => ['string'],
5960
'language' => ['string', 'max:15', 'alpha_dash'],
60-
'password' => [Password::default()],
61+
'password' => ['string', Password::default()],
6162
'roles' => ['array'],
6263
'roles.*' => ['integer'],
6364
],

dev/api/responses/users-list.json

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
"updated_at": "2022-02-03T16:27:55.000000Z",
99
"external_auth_id": "abc123456",
1010
"slug": "dan-brown",
11-
"user_id": 1,
1211
"last_activity_at": "2022-02-03T16:27:55.000000Z",
1312
"profile_url": "https://docs.example.com/user/dan-brown",
1413
"edit_url": "https://docs.example.com/settings/users/1",
@@ -22,7 +21,6 @@
2221
"updated_at": "2021-11-18T17:10:58.000000Z",
2322
"external_auth_id": "",
2423
"slug": "benny",
25-
"user_id": 2,
2624
"last_activity_at": "2022-01-31T20:39:24.000000Z",
2725
"profile_url": "https://docs.example.com/user/benny",
2826
"edit_url": "https://docs.example.com/settings/users/2",

0 commit comments

Comments
 (0)