Compatibilite crudit studio - #29
Conversation
…ntialBundle into compatibilite_crudit_studio
| sectionCheckbox.checked = groupCheckbox.checked; | ||
| }); | ||
|
|
||
| showToast("Success", "#1CC88A"); |
There was a problem hiding this comment.
le message est toujours en anglais ?
| } | ||
|
|
||
| let groupId = groupCheckbox.dataset.groupId; | ||
| fetch('/admin/credential/toggle-group/' + groupId + '/' + (groupCheckbox.checked ? 1 : 0), { method: 'post' }) |
There was a problem hiding this comment.
du coup on part du principe qu'on met /admin en préfixe. bon j'imagine que c'était déjà le cas en v1
| 'POST', | ||
| $this->clientUrl . '/api/project/init/' . $this->projectCode, | ||
| [ | ||
| 'headers' => [ |
There was a problem hiding this comment.
on peut faire mieux avec les options json et auth_bearer. et JSON_PRETTY_PRINT ne sert à rien pour une api
| $cred->setRole($role); | ||
| $cred->setRubrique(""); | ||
| } | ||
| $credential = new Credential(); |
There was a problem hiding this comment.
plus de gestion de doublon ? vous êtes sûrs ?
There was a problem hiding this comment.
d'ailleurs pareil pour les groupes ?
There was a problem hiding this comment.
Non plus de doublon que ce soit au niveau des credentials ou des groupes
There was a problem hiding this comment.
Soit il va récupérer l'existant et le mettre à jour, soit il va le créer
| $clientUrl = $this->parameterBag->get('lle_credential.client_url'); | ||
| $projectCode = $this->parameterBag->get('lle_credential.project_code'); | ||
|
|
||
| $response = $this->client->request( |
There was a problem hiding this comment.
$this->client n'existe pas
|
|
||
| public function load(): void | ||
| { | ||
| $clientUrl = $this->parameterBag->get('lle_credential.client_url'); |
There was a problem hiding this comment.
je suggère de faire un service qui fait office de client et qui gère l'url tout seul. c'est trop verbeux et répétitif sinon.
|
|
||
| #[AsCommand( | ||
| name: 'credential:warmup', | ||
| name: 'lle:credential:warmup', |
There was a problem hiding this comment.
faudra préciser qu'il faut changer les Makefile.
|
|
||
| $response = $this->credentialService->sendCredentials($results); | ||
| $response = $this->credentialService->sendCredentials(); | ||
| // if ($response === 200) { |
| } | ||
|
|
||
| public function setLabel(string $label): self | ||
| public function setLabel(string $lable): self |
| { | ||
| $data = []; | ||
| /** @var array $locales */ | ||
| $locales = $this->parameterBag->get('locales'); |
There was a problem hiding this comment.
hum du coup on rajoute cette dépendance à l'existence du paramètres locales.
3 participants
No description provided.