KLYMM / cnergy-cms

0 stars 4 forks source link

[FE Menu] Enhancement Add/Edit FE Menu #433

Open leonardochristofer opened 1 year ago

leonardochristofer commented 1 year ago

Description

  1. Mengubah field position menjadi checkbox (Header & Footer)

  2. Menambahkan field:

Ada kondisi yang harus terpenuhi merefer dari Newshub, yaitu:

Image

anggagilang11 commented 1 year ago

Mbak @chusnulnabila minta tolong ditambahin panjang data slugnya, jadi 150 kayaknya aman. Thank you

Image

leonardochristofer commented 1 year ago

slugnya udh diupdate ya

anggagilang11 commented 1 year ago

woke thank you

anggagilang11 commented 1 year ago

Mbak @chusnulnabila minta tolong untuk format return urlnya disamain ya dengan yang Newshub. Aku udah masukin link url yang sama, tapi nilainya ketika disimpan berbeda.

Image

Sama untuk targetnya, kalau di CMS opsi yang dipilih

Image

Terima kasih

leonardochristofer commented 1 year ago

@anggagilang11 next time tolong dibikin task baru aja ya, apa lagi ini kan masalah API return dan url type target ini kan feature yang baru di develop, harusnya untuk perubahan API ada task baru sih. tp yg kali ini gpp.

Sebelumnya type target url di hardcode karena blm ada featurenya.

anggagilang11 commented 1 year ago

noted, iya sih harus masuk task API fe menu, thank you @leonardochristofer

leonardochristofer commented 1 year ago

@anggagilang11 btw url yang sebelumnya kan /nama-menu cuma kurang ditambah domain aja, yg skrg mau dibuat bebas edit atau tetep domain/nama-menu

Sebelumnya nda ada inputan url sih, auto sama ke nama menunya

Image

anggagilang11 commented 1 year ago

kalau untuk url yang berisi slug nama-menu perlu ditambahin domain, tapi kalau url yang bentuknya full url misalnya https://cms-mm.kl-youniverse.com/master/category atau link eksternal seperti gak perlu domain takutnya bentrok linknya. Kira-kira gimana ya? apa perlu dibedakan jadi ada kondisi buat bedain nilai tambahannya tapi return di API tetep satu "url" @leonardochristofer

Ini kalau dilihat dari API Newshub type

image

leonardochristofer commented 1 year ago

@anggagilang11 discuss ke mentor aja untuk masalah ini, tapi kalau menurutku url ini nda perlu input sih seperti sebelumnya karena kalau input kan bisa nda sesuai sama nama menu dan bisa typo.

Untuk sementara, aku kasih domain/nama-menu dulu, nanti kalau ada perubahan lain termasuk yang # atau / atau null bisa move ke task yang baru aja, yang ini kalau bisa di move ke done dulu karena scopenya sudah terlalu jauh dari task awal. thank you.