News view refactor #137

Merged
jsegarra merged 4 commits from wbuezas/hedera-web-mindshore:feature/news-view into beta 2025-04-04 11:09:56 +00:00
Member

PR relacionada salix: verdnatura/salix#3638

PR relacionada salix: https://gitea.verdnatura.es/verdnatura/salix/pulls/3638
wbuezas added 1 commit 2025-04-03 07:50:39 +00:00
gitea/hedera-web/pipeline/pr-beta This commit looks good Details
6cb8267a0b
News view refactor
wbuezas requested review from jsegarra 2025-04-03 07:50:46 +00:00
jsegarra added 1 commit 2025-04-03 08:29:59 +00:00
gitea/hedera-web/pipeline/pr-beta This commit looks good Details
28b375cbbb
Merge branch 'beta' into feature/news-view
jsegarra requested changes 2025-04-03 08:40:41 +00:00
Dismissed
@ -86,3 +83,1 @@
table="news"
schema="hedera"
separation-between-inputs="lg"
:url="initialFetchUrl"
Member

Esto hay que hacerlo por cada componente que use FormModel?
si es una validación tan corta, no podriamos hacerlo dentro de FormModel?

Esto hay que hacerlo por cada componente que use FormModel? si es una validación tan corta, no podriamos hacerlo dentro de FormModel?
Author
Member

Como lo validarias dentro del FormModel?
Asi es como funciona actualmente en el FormModel de Lilium, si se pasa un urlCreate al guardar dispara un post y si no dispara un patch, y la prop url si se manda significa que FormModel deberia fetchear la data con la url que se provee

Como lo validarias dentro del FormModel? Asi es como funciona actualmente en el FormModel de Lilium, si se pasa un urlCreate al guardar dispara un post y si no dispara un patch, y la prop `url` si se manda significa que FormModel deberia fetchear la data con la url que se provee
@ -89,0 +84,4 @@
:auto-load="isEditMode"
:url-create="urlCreate"
:url-update="urlUpdate"
:data-required="{ userFk: userStore.userId }"
Member

esta propiedad es porque el valor nunca debe eliminarse o es que el formulario debe tenerlo de inicio¿?

esta propiedad es porque el valor *nunca* debe eliminarse o es que el formulario debe tenerlo de inicio¿?
Author
Member

Porque es un valor que siempre es requerido tanto para update o create

Porque es un valor que siempre es requerido tanto para update o create
@ -24,28 +25,19 @@ const news = ref([]);
const getNews = async () => {
Member

Seria lo mismo que usar FetchData, no?
Y si le ponemos un metodo de onfetch dentro de VnList?

Seria lo mismo que usar FetchData, no? Y si le ponemos un metodo de onfetch dentro de VnList?
Author
Member

Aplique el componente de FetchData

Commit: 824fa28126

Aplique el componente de `FetchData` Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/824fa281265029b5d2940c68510d5586f5930c99
wbuezas added 2 commits 2025-04-04 09:50:13 +00:00
jsegarra approved these changes 2025-04-04 11:09:51 +00:00
jsegarra merged commit be54a93e63 into beta 2025-04-04 11:09:56 +00:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/hedera-web#137
No description provided.