generated from Code-Institute-Org/gitpod-full-template
-
Notifications
You must be signed in to change notification settings - Fork 1
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.
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
USER STORY: Update all tabs' content & counts when tasks are watched/unwatched #124
Closed
11 tasks done
Labels
Milestone
Comments
blahosyl
added
p: could have
Priority: Nice to have
e: ux tasks
Epic: UX for Tasls
labels
Aug 13, 2024
blahosyl
added a commit
that referenced
this issue
Aug 14, 2024
blahosyl
added a commit
that referenced
this issue
Aug 14, 2024
move `changeWatch` to `TaskTabs & pass it down the component tree
blahosyl
added a commit
that referenced
this issue
Aug 14, 2024
… when tasks are watched/unwatched #124
blahosyl
changed the title
USER STORY: Update tab counts when tasks are watched/unwatched
USER STORY: Update all tabs' content & counts when tasks are watched/unwatched
Aug 14, 2024
blahosyl
added a commit
that referenced
this issue
Aug 14, 2024
blahosyl
added a commit
that referenced
this issue
Aug 14, 2024
blahosyl
added a commit
that referenced
this issue
Aug 14, 2024
works but it takes a while to settle down, and the number goes back and forth
blahosyl
added
p: should have
Priority: High value but not crucial
and removed
p: could have
Priority: Nice to have
labels
Aug 14, 2024
This was referenced Aug 14, 2024
blahosyl
added a commit
that referenced
this issue
Aug 15, 2024
1 task
Further streamlining of the code might be possible #132 |
github-project-automation
bot
moved this from In Progress
to Done/Won't do
in Task Manager
Aug 15, 2024
blahosyl
added a commit
that referenced
this issue
Aug 15, 2024
blahosyl
added a commit
that referenced
this issue
Aug 16, 2024
after the restructuring of `TaskKanban` to `TaskList`, we also need to pass the `taskTlist` prop to `TaskList` in `ProfileDetail`
This was referenced Aug 16, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
As a Logged-in User,
I want to see the count of tasks I'm watching be updated in real time,
so that can be informed of the change without reloading the page.
Acceptance Criteria
When tasks are watched/unwatched in other tabs,
is updated without reloading the page
Tasks
Update tasks in all columns when a task is watched/unwatched
changedWatch
&setChangedWatch
state inTaskTabs
&ProfileDetail
TaskList
/TaskKanban
on all tabschangedWatch
&setChangedWatch
inTaskList
/TaskKanban
, pass it down toTask
TaskList
/TaskKanban
inuseEffect()
, addchangedWatch, setChangedWatch
to the triggers of the functionTaskList
/TaskKanban
inuseEffect()
, addsetChangedWatch(false);
undersetHasLoaded(false);
changedWatch
&setChangedWatch
inTask
Task
, addsetChangedWatch(true);
tohandleWatch
&handleUnwatch ()
ProfileDetail
, also get the list of tasks withfetchData
Update the counts of all watched columns when
Implemented using the
fetchData
function fromProfileDetail
, rewritten asfetchProfileData
inTaskList
. However,fetchData
is also used inTaskTab
, so maybe this implementation would be preferred.ProfileDetail
TaskKanban
OR move rendering logic from TaskKanban to TaskDetail, and render list or kanban conditionally depending on thetaskList
prop within theTaskList
componentThe text was updated successfully, but these errors were encountered: