fix(floating-rates): unique date picker ids and bindable datetime buttons in @for rows - #557
cestercian wants to merge 2 commits into
Conversation
Period rows created by Add Period shared one picker id and the ion-datetime-button initialized before its ion-datetime existed. Defer each button relative to its own row and give every picker a unique id. Fixes apache#548
ion-datetime cannot parse a Date's locale string, so a bound picker still rendered blank. Store each period's fromDate as YYYY-MM-DD.
Commits on this pull request are not verified2 of 2 commits are not showing as Verified on GitHub, so this pull request cannot be merged into
|
E2E — mocked backend🎭 E2E Tests✅ All green — 362 passed · 0 failed · 1 skipped, across 30 spec files in 6m 5s. By spec file
All 363 tests — click to expand
Slowest 10 — what the shard counts should be tuned against
📼 Download the HTML report, videos and traces — see the Generated by run 35469403594 from |
E2E — real Fineract🎭 E2E Tests✅ All green — 78 passed · 0 failed · 0 skipped, across 24 spec files in 5m 51s. By spec file
All 78 tests — click to expand
Slowest 10 — what the shard counts should be tuned against
📼 Download the HTML report, videos and traces — see the Generated by run 35469403594 from |
Summary
Date pickers created inside
@forshared one ID and never bound. Each rate-period row now gets a unique picker id, and a deferredion-datetime-buttonwaits until its picker is in the DOM before stampingdatetime. Period dates are stored asYYYY-MM-DDso the bound button shows a date.Fixes #548
Test plan