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.

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

Staff can view checklist on pets show - tasks tab and interact with it #257

Closed
kasugaijin opened this issue Oct 11, 2023 · 9 comments
Closed
Labels
help wanted Extra attention is needed

Comments

@kasugaijin
Copy link
Collaborator

Blocked by: #256

As a staff member I can navigate to a pets show page, tasks tab, and see a checklist of items. I can add, edit, delete and mark complete each item.

Acceptance criteria

  • visible on pets show, tasks tab
  • allows CRUD of tasks
  • task can be marked complete, with visual queue
  • use reasonable dashboard template for this purpose (reach out to team on Slack if you have questions here)

Theme snippets can be gotten from https://geeksui.codescandy.com/geeks/docs/index.html and theme https://geeksui.codescandy.com/geeks/index.html

@ErinClaudio
Copy link
Collaborator

@kasugaijin Taking this one unless told otherwise.

@ErinClaudio ErinClaudio self-assigned this Oct 11, 2023
@ErinClaudio
Copy link
Collaborator

Screenshot 2023-10-11 at 2 13 21 PM Screenshot 2023-10-11 at 2 13 40 PM

found this error when clicking on Task
I am working on resolving this now

@ErinClaudio
Copy link
Collaborator

ErinClaudio commented Oct 11, 2023

@kasugaijin
I've come across a visibility issue on the pet's show page where clicking between tabs (Overview, Task, Applications, Files) doesn't toggle the visibility of corresponding sections; all sections remain visible at once. After reviewing possible solutions, I'm leaning towards utilizing Rails partial views to load the relevant section dynamically based on the selected tab. Would you prefer this approach, or is there another method you'd recommend?

@kasugaijin
Copy link
Collaborator Author

kasugaijin commented Oct 11, 2023

@ErinClaudio
FYI I assigned you to the precursor ticket #256 yesterday...that's the background for the checklist work. My bad for not commenting and @ing you on there. I thought assigning would notify.

I am thinking this ticket #242 is part of the solution to the partials piece. I agree with your initial approach.

How do we want to tackle this? My thinking is the work is to be done in this order:
#242, then
#256, then
#257

Thanks!

@ErinClaudio
Copy link
Collaborator

@kasugaijin Then I will assign myself, start there, and try to work through them. Does that sound reasonable?

@kasugaijin
Copy link
Collaborator Author

Sounds good. Thank you @ErinClaudio

@nsiwnf
Copy link
Collaborator

nsiwnf commented Oct 14, 2023

Was also thinking this might be a good layout for tasks
The columns here could be ChecklistTemplates, each card a ChecklistItem, and checked/unchecked are the ChecklistAssignments
https://geeksui.codescandy.com/geeks/pages/dashboard/task-kanban.html

Copy link

This issue is marked as stale due to no activity within 30 days. If no further activity is detected within 7 days, it will be unassigned.

Copy link

Automatically unassigned after 7 days of inactivity.

@nsiwnf nsiwnf linked a pull request Dec 8, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants