Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

enhance work tracker to use the new getCards #1966

Merged
merged 13 commits into from
Dec 23, 2024
Merged

enhance work tracker to use the new getCards #1966

merged 13 commits into from
Dec 23, 2024

Conversation

tintinthong
Copy link
Contributor

@tintinthong tintinthong commented Dec 21, 2024 •
edited
Loading

Changes

  • uses getCard resource at the root of the component. The data is loaded first and never reloaded.
  • we no longer track sort order after changing the filter rather we just hide the cards. So the kanban actually knows of all the cards but hides them based upon the filter

Things to test

@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 21, 2024 •
edited
Loading

Host Test Results

    1 files  ±0      1 suites  ±0   20m 12s ⏱️ -19s
718 tests ±0  716 ✔️ ±0  2 💤 ±0  0 ❌ ±0 
723 runs  ±0  721 ✔️ ±0  2 💤 ±0  0 ❌ ±0 

Results for commit 9803dd3. ± Comparison against base commit 884e3ed.

♻️ This comment has been updated with latest results.

@tintinthong tintinthong force-pushed the eco-135-integrate-new-get-cards branch from d5ae2da to 25a77c6 Compare December 21, 2024 23:11
@richardhjtan richardhjtan merged commit 69f58f4 into main Dec 23, 2024
51 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@richardhjtan richardhjtan richardhjtan approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tintinthong @richardhjtan