ariasat/agecem-org
2
1
Fork 0
forked from agecem/agecem-org
Code Issues 16 Pull requests 4 Packages Activity Actions

opts.go et pagination + search pour list blogues #42

Merged
dnova merged 12 commits from opts into main 2026-07-06 17:45:57 -04:00
Conversation 4 Commits 12 Files changed 12 +373 -22
ariasat commented 2026-07-03 20:09:03 -04:00
Owner
Copy link

closes #37
closes #38

closes #37 closes #38
refactor: ui responses dans response_ui.go
All checks were successful
/ build (pull_request) Successful in 1m24s
Details
/ help (pull_request) Successful in -6s
Details
442c151194
feature: ui pour pagination de blogues
All checks were successful
/ build (pull_request) Successful in 1m15s
Details
/ help (pull_request) Successful in -6s
Details
ccd08315c9
Merge branch 'main' into opts
All checks were successful
/ build (pull_request) Successful in 1m8s
Details
/ help (pull_request) Successful in -6s
Details
f4e4b621ac
ariasat changed title from WIP: opts.go et pagination + search pour list blogues to opts.go et pagination + search pour list blogues 2026-07-03 20:57:08 -04:00
Merge branch 'main' into opts
All checks were successful
/ build (pull_request) Successful in 1m7s
Details
/ help (pull_request) Successful in -6s
Details
a015b215f5
dnova reviewed 2026-07-06 16:08:45 -04:00
handler_ui.go
@ -15,3 +16,4 @@
"github.com/google/uuid"
)
const ROWS_LIMIT = 20
dnova commented 2026-07-06 16:08:45 -04:00
Collaborator
Copy link

cela nest pas lié au offset du back alors la vue que les deux se ressemblent ca va marchermais si disont on a 10 dans le rows_limit ca va pas marcher , il faut lui passer dans le withlimit et withofset a chaque fois comme dans cms.
image

cela nest pas lié au offset du back alors la vue que les deux se ressemblent ca va marchermais si disont on a 10 dans le rows_limit ca va pas marcher , il faut lui passer dans le withlimit et withofset a chaque fois comme dans cms. ![image](/attachments/e25bb4b3-8ddc-479b-9c05-78cb0a672807)
image.png
195 KiB
dnova commented 2026-07-06 16:09:45 -04:00
Collaborator
Copy link

image

![image](/attachments/22bfa357-4ab9-42b3-b1bb-0a7e806616b1)
image.png
28 KiB
ariasat marked this conversation as resolved
dnova reviewed 2026-07-06 16:14:42 -04:00
handler_ui.go Outdated
@ -159,0 +178,4 @@
offset := (page - 1) * ROWS_LIMIT
opts := []ListOption{
WithOffset(&offset),
dnova commented 2026-07-06 16:14:42 -04:00
Collaborator
Copy link

il manque le withlimit:
limit := ROWS_LIMIT

	opts := []ListOption{
		WithLimit(&limit),
		WithOffset(&offset),
		WithSearch(q.Get("search")),
	}
il manque le withlimit: limit := ROWS_LIMIT opts := []ListOption{ WithLimit(&limit), WithOffset(&offset), WithSearch(q.Get("search")), }
ariasat marked this conversation as resolved
dnova reviewed 2026-07-06 16:18:03 -04:00
handler_ui.go Outdated
@ -270,0 +318,4 @@
}
offset := (page - 1) * ROWS_LIMIT
res, err := client.ListBlogues(h.Ctx, WithOffset(&offset))
dnova commented 2026-07-06 16:18:03 -04:00
Collaborator
Copy link

Withlimit encore ici apres

Withlimit encore ici apres
ariasat marked this conversation as resolved
fix: listBlogues avec withlimit dans ui handler
All checks were successful
/ build (pull_request) Successful in 1m28s
Details
/ help (pull_request) Successful in -6s
Details
d7152a36e1
dnova reviewed 2026-07-06 17:13:31 -04:00
handler_ui.go Outdated
@ -159,3 +200,4 @@
res, err := client.ListBlogues(h.Ctx, opts...)
if err != nil {
data.Error = friendlyError(err)
}
dnova commented 2026-07-06 17:13:31 -04:00
Collaborator
Copy link

cela devrais etre pageSize non? parce que cest le total mais dans la page seulement non?

cela devrais etre pageSize non? parce que cest le total mais dans la page seulement non?
ariasat commented 2026-07-06 17:25:04 -04:00
Author
Owner
Copy link

je vois pas le code que tu reference

res, err := client.ListBlogues(h.Ctx, opts...)
			if err != nil {
				data.Error = friendlyError(err)
			}
je vois pas le code que tu reference ``` res, err := client.ListBlogues(h.Ctx, opts...) if err != nil { data.Error = friendlyError(err) } ```
dnova commented 2026-07-06 17:30:15 -04:00
Collaborator
Copy link

data.Total = len(res.Data.Blogues):
ligne 205 handler_ui

data.Total = len(res.Data.Blogues): ligne 205 handler_ui
ariasat marked this conversation as resolved
Merge branch 'main' into opts
All checks were successful
/ build (pull_request) Successful in 1m35s
Details
/ help (pull_request) Successful in -7s
Details
a8b275a172
fix: total -> pageSize
All checks were successful
/ build (pull_request) Successful in 1m8s
Details
/ help (pull_request) Successful in -7s
Details
dbb908e88f
dnova merged commit 1c67aaeaec into main 2026-07-06 17:45:57 -04:00
dnova deleted branch opts 2026-07-06 17:46:00 -04:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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
ariasat/agecem-org!42
No description provided.