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

As staff, I want to be able to see the applications for a pet from the pet info page #258

Closed
nsiwnf opened this issue Oct 11, 2023 · 6 comments · Fixed by #288
Closed

As staff, I want to be able to see the applications for a pet from the pet info page #258

nsiwnf opened this issue Oct 11, 2023 · 6 comments · Fixed by #288
Assignees

Comments

@nsiwnf
Copy link
Collaborator

nsiwnf commented Oct 11, 2023

As staff, when I click the "Applications" tab, I should see a list of applications open for this pet (where "Summary is for "Overview")

Screen Shot 2023-10-10 at 6 16 43 PM
@aisayo
Copy link
Contributor

aisayo commented Oct 11, 2023

Hi, I would like to work on this. Can you please assign to me

@kasugaijin
Copy link
Collaborator

All yours! @aisayo
Note that @ErinClaudio working on #257 has an approach to use to render the required template when clicking the Applications tab. Would be good if we're consitent.

@kasugaijin
Copy link
Collaborator

kasugaijin commented Oct 13, 2023

FYI PR #265 is a pre-cursor for setting up nice partials and will set the approach for this ticket.

@aisayo
Copy link
Contributor

aisayo commented Oct 17, 2023

Hello @kasugaijin , should tests be written as well?

@nsiwnf
Copy link
Collaborator Author

nsiwnf commented Oct 17, 2023

@aisayo @kasugaijin Yes, I think so! Take a look at #275 for an example if needed

@aisayo
Copy link
Contributor

aisayo commented Oct 19, 2023

Hey! Do you prefer I base the PR for this off Staff-can-view-checklist-on-pets-show-and-interact or main?

@nsiwnf nsiwnf linked a pull request Oct 24, 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
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants