900 - AdopterFoster: Add adopted pets

Open sarvaiyanidhi opened this issue 1 year ago • 2 comments

🔗 Issue

#900

✍️ Description

Displaying Adopted Pets for Adopter account and listing its files on click of each pet.

📷 Screenshots/Demos

Alta-Pet-Rescue-Adopted-Pets-

Aug 02 '24 02:08 sarvaiyanidhi

@sarvaiyanidhi thanks for doing this, one of us will review this soon.

Aug 14 '24 00:08 kasugaijin

@sarvaiyanidhi great start on this! The functionality is all good. I have some comments about structure and approach in some cases. Let me know if you have any questions on these comments :)

Thanks @kasugaijin for the detailed feedback. Have gone through the comments and will push the required changes soon.

Aug 20 '24 00:08 sarvaiyanidhi

@sarvaiyanidhi how's this going?

Aug 28 '24 00:08 kasugaijin

@sarvaiyanidhi how's this going?

Hey sorry for the delay... Will work and push changes in 1-2 days

Aug 28 '24 02:08 sarvaiyanidhi

@sarvaiyanidhi not at all! No rush. Just making sure you’re not blocked.

Aug 28 '24 02:08 kasugaijin

Hi @kasugaijin,

Sorry took some more time than expected as was tied up with few things.. I have addressed all points and pushed code. Can you please check if changes made for query using Match model is as per requirement. Also, I am facing one issue with authorize test in controller test, not able to figure why it is failing if you can help with it.

Thanks

Sep 04 '24 20:09 sarvaiyanidhi

Hi @kasugaijin,

Sorry took some more time than expected as was tied up with few things.. I have addressed all points and pushed code. Can you please check if changes made for query using Match model is as per requirement. Also, I am facing one issue with authorize test in controller test, not able to figure why it is failing if you can help with it.

Thanks

Thanks @sarvaiyanidhi. Can you post a picture of the error screenshot in the pet rescue slack channel? It's the fastest way to get unblocked.

Sep 04 '24 21:09 kasugaijin