Lio Novelli lio
Loading Heatmap…

lio commented on issue yufu/manifest#29

Obvescanje prek email

WIP: PR #42

2024-01-23 23:25:43 +01:00

lio created pull request yufu/manifest#42

Obvescanje prek emailov.

2024-01-23 23:25:05 +01:00

lio created branch mail-obvescanje in yufu/manifest

2024-01-23 23:23:32 +01:00

lio pushed to mail-obvescanje at yufu/manifest

2024-01-23 23:23:32 +01:00

lio commented on pull request yufu/manifest#26

Obrazec za dodajanje in urejanje pojma

for test/non-production instances it would be good if wrong key would pass validation, I guess

2024-01-23 20:59:50 +01:00

lio approved yufu/manifest#26

Obrazec za dodajanje in urejanje pojma

Nekaj lepotnih komentarjev in nekaj nerazumljenih vrstic.

2024-01-23 20:59:50 +01:00

lio commented on pull request yufu/manifest#26

Obrazec za dodajanje in urejanje pojma

a bi bilo treba nastavit $requirements spremenljivko tudi ce je request uspesen in vrnit nek pozitiven rezultat?

2024-01-23 20:59:49 +01:00

lio commented on pull request yufu/manifest#26

Obrazec za dodajanje in urejanje pojma

zakaj se imenuje example?

2024-01-23 20:59:49 +01:00

lio commented on pull request yufu/manifest#26

Obrazec za dodajanje in urejanje pojma

we could probably enable this and allow necessary domains

2024-01-23 20:59:49 +01:00

lio commented on pull request yufu/manifest#26

Obrazec za dodajanje in urejanje pojma

tukaj

2024-01-23 20:59:49 +01:00

lio commented on pull request yufu/manifest#26

Obrazec za dodajanje in urejanje pojma

mal sumljivo

2024-01-23 20:59:49 +01:00

lio commented on pull request yufu/manifest#26

Obrazec za dodajanje in urejanje pojma

Ni kul, da je defaultni jezik hrvaski, ali pac?

2024-01-23 20:59:49 +01:00

lio commented on pull request yufu/manifest#26

Obrazec za dodajanje in urejanje pojma

to sem ugotovil, da je anti-pattern in se ne dela

2024-01-23 20:59:49 +01:00

lio commented on pull request yufu/manifest#26

Obrazec za dodajanje in urejanje pojma

missing comment

2024-01-23 20:59:49 +01:00

lio commented on pull request yufu/manifest#26

Obrazec za dodajanje in urejanje pojma

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

2024-01-23 20:59:49 +01:00

lio commented on pull request yufu/manifest#26

Obrazec za dodajanje in urejanje pojma

Wrong comment.

2024-01-23 20:59:49 +01:00

lio commented on pull request yufu/manifest#26

Obrazec za dodajanje in urejanje pojma

zdi se mi, da bi bilo dobr pogledat, ce prefix ze ni dodan, da ne pride do podvajanja

2024-01-23 20:59:49 +01:00

lio commented on issue yufu/manifest#31

Povezani koncepti na dodaj pojem endpointu

Boljsa ideja: - naredimo etherpad plugin, ki deluje kot node_insert na drupalu. - presave hook na node-u, ki pregleda tekst, poisce povezane prispevke in jih doda v polje related conecepts

2024-01-14 21:49:17 +01:00

lio closed pull request yufu/manifest#30

Content view (genericen)

2024-01-14 19:49:13 +01:00

lio approved yufu/manifest#30

Content view (genericen)

zgleda ok

2024-01-14 19:48:58 +01:00