Closed Th1nkK1D closed 1 month ago
@JaeAiKay ฝากอันนี้ไว้ในช่วงที่เราไม่อยู่จนถึงวันที่ 5 ส.ค. ลองอ่านดู ทำเท่าที่ได้ ตรงไหนติดเดี๋ยวกลับมาช่วย ละในฐานะที่ทำมาหลาย PR ละเราฝากบรีฟวิธีการทำงาน และแบ่งงานใน issue นี้กับ @napatswift หน่อย (บ่นอยากทำงาน เรา invite เข้า repo ไปแล้วโปรด accept) ทำใน branch เดียวกันเลยก็ได้ละเปิด PR มาทีเดียว
ฝาก assign ชื่อตัวเองเข้า issue นี้ทั้งคู่เลย กราบขอบพระคุณเป็นอย่างสูง
รับแซ่บบบ
@Th1nkK1D @JaeAiKay มีคำถามเรื่อง tab หน่อยครับ เท่าที่ผมลองดูใน repo นี้ในหน้า assemblies/[id]/members/
มีการใช้ tab แต่ไม่ใช่การเปลี่ยน tab โดยการใช้ state แต่เป็นการเปลี่ยน path ไปที่ tab view นั้น เลยอยากรู้ว่าอยากให้ทำท่าเดียวกันดีไหมครับ
@Th1nkK1D @JaeAiKay มีคำถามเรื่อง tab หน่อยครับ เท่าที่ผมลองดูใน repo นี้ในหน้า
assemblies/[id]/members/
มีการใช้ tab แต่ไม่ใช่การเปลี่ยน tab โดยการใช้ state แต่เป็นการเปลี่ยน path ไปที่ tab view นั้น เลยอยากรู้ว่าอยากให้ทำท่าเดียวกันดีไหมครับ
ในกรณีนี้ดีไซน์อยากให้ทั้ง 2 แท็บและ control ข้างบน sync กันหมด สมาชิกในแท็บแรกต้องเป็นของ ณ วันที่เลือก และวันนั้นต้องไฮไลท์ในไทม์ไลน์การเปลี่ยนแปลงในแท็บที่สอง มองซะว่าเป็น 2 view ที่แสดงมาจาก state เดียวกัน เพราะงั้นเราแยกหน้าบ่ได้ เปลี่ยนแท็บแล้ววันที่เลือกจะหาย
ถ้าเราเก็บทั้ง tab ทั้งวันให้อยู่ใน path หรือ url params ก็ไม่ได้ใช่ไหมครับ เหมือนมันจะต้อง generate path ตอน build (รึเปล่า)
ถ้าเราเก็บทั้ง tab ทั้งวันให้อยู่ใน path หรือ url params ก็ไม่ได้ใช่ไหมครับ เหมือนมันจะต้อง generate path ตอน build (รึเปล่า)
ใช่แล้ว คิดว่าไม่ต้องทำอะไรเกี่ยวกับ path/param เลยสำหรับ issue นี้ เป็นหน้าเดียวไปได้เลยแล้วเก็บ state เอา
สุดแจ่ม
This is a huge issue and requires some experimentation. Please read carefully.
View Figma file for more detail brief
Notes
src/routes/assemblies/[id]/changes/+page.svelte
. Currently has no UI, just the data prepared by adjacent+page.server.ts
assembly
= assembly infoavailableAssemblies
= list of all assemblies to create a navigation linkassemblyRoles
= list of every combination of politicians (id) and their role in the current assembly (one politician can have more than one role. This is the correct behavior. The member list in the main assembly page is incorrect since current logic removes duplicated politicians, but fixing it is out of this issue's scope)members
politician information that exists inassemblyRoles
/assemblies/คณะรัฐมนตรี-63/changes
for testing.