Obrazec za dodajanje in urejanje pojma #26
No reviewers
Labels
No Label
bug
duplicate
enhancement
help wanted
invalid
Needs Review
Potrebna dodatna obrazložitev
question
task
wontfix
Zelo Pomembno
No Milestone
No project
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: yufu/manifest#26
Loading…
Reference in New Issue
There is no content yet.
Delete Branch "nuxt-pojem-form"
Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?
Z etherpad integracijo.
WIP!
V .install fajlu je hook scheme nepotreben, ni treba tabele ustvarjat.
V settinga formi pa mislim, da se za key lahko uporabi tip polja password kot je narejeno tukaj https://git.drupalcode.org/project/ohdear_integration/-/blob/1.x/src/Form/SettingsForm.php?ref_type=heads#L53
Obrazec za urejanje pojmato Obrazec za dodajanje in urejanje pojmaUrejeno, sem dodal service, requirement in izboljšal obrazec za nastavitve
Obrazec za dodajanje in urejanje pojmato WIP: Obrazec za dodajanje in urejanje pojmaea1f55149e
toef936da4bb
ef936da4bb
to33da93a1dd
WIP: Obrazec za dodajanje in urejanje pojmato Obrazec za dodajanje in urejanje pojmaNekaj lepotnih komentarjev in nekaj nerazumljenih vrstic.
@ -1,3 +1,3 @@
_core:
default_config_hash: 2OMXCScXUOLSYID9-phjO4q36nnnaMWNUlDxEqZzG1U
use_admin_theme: true
use_admin_theme: false
mal sumljivo
@ -9,3 +9,3 @@
default_config_hash: 12Bd0mJQFIaXAkRfMVCAAcZ0oaxm94PoK8oHR9hkLmY
id: content
label: Vsebine
label: Sadržaj
Ni kul, da je defaultni jezik hrvaski, ali pac?
@ -0,0 +22,4 @@
} catch (ClientException $exception) {
$value = $exception->getCode();
$msg = $exception->getMessage();
$requirements['etherpad_api_status'] = [
a bi bilo treba nastavit $requirements spremenljivko tudi ce je request uspesen in vrnit nek pozitiven rezultat?
Je nastavljen višje, če prav razumem, je to samo za feedback če kaj ne štima.
@ -0,0 +1,15 @@
etherpad_api.example:
zakaj se imenuje example?
Ne se ne. Več.
@ -0,0 +3,4 @@
class: Drupal\etherpad_api\PathProcessor\EtherpadAPIPathProcessor
tags:
- { name: path_processor_inbound, priority: 1000 }
etherpad_api.settings:
to sem ugotovil, da je anti-pattern in se ne dela
https://mglaman.dev/blog/dependency-injection-anti-patterns-drupal
Baje ni grozno in bom pustil, hvala za info za naprej.
@ -0,0 +40,4 @@
*/
public function __construct(ClientInterface $httpClient, ImmutableConfig $config) {
$this->httpClient = $httpClient;
$this->config = $config;
Naceloma ok, ampak config kot property zna bit kdaj rezerviran za kaj drugega in je treba bit pazljiv. Bolje je poimenovat direktno - etherpadApiConfig. oziroma itq je to antipattern in bi bilo treba tukaj injectat config.factory in sele v kodi spodaj dobit vrednosti konfiga
Sem ga kar izbacil.
@ -0,0 +45,4 @@
$this->apiKey = $config->get('key');
}
public function checkToken() {
missing comment
@ -0,0 +53,4 @@
}
/**
* Method description.
Wrong comment.
@ -0,0 +36,4 @@
$data = array_merge($request->query->all(), $request->request->all());
if ($data['padID']) {
// @TODO prefix v config!
$data['padID'] = 'yufu-' . $data['padID'];
zdi se mi, da bi bilo dobr pogledat, ce prefix ze ni dodan, da ne pride do podvajanja
@ -0,0 +72,4 @@
$url = rtrim($baseUrl, '/') . "/1.2/checkToken?apikey=$apiKey";
try {
$this->httpClient->request('get', $url);
for test/non-production instances it would be good if wrong key would pass validation, I guess
@ -0,0 +206,4 @@
# for more information about the topic in general.
# Note: By default the configuration is disabled.
cors.config:
enabled: false
we could probably enable this and allow necessary domains
@ -0,0 +2,4 @@
cors.config:
enabled: true
allowedHeaders: ['x-csrf-token','authorization','content-type','accept','origin','x-requested-with','access-control-allow-origin','x-allowed-header','*']
allowedOrigins: ['*']
tukaj