Display feature pets on org home page [558]

[VonTeacher]

🔗 Issue

Display feature pets on org home page #558

✍️ Description

• [x] Created instance variable in the Organizations::HomeController#index action to sample from the adoptable_pets (though I'm not sure the pattern matches what is used in other controllers).
• [x] Replaced four static pet cards on the /<org>/home page with a sample of the adoptable pets from that organization.

📷 Screenshots/Demos

Before:

After:

[kasugaijin]

FYI tests failures and need to run the linter.

[kasugaijin]

@all-contributors please add @VonTeacher for code.

[allcontributors[bot]]

@kasugaijin

I've put up a pull request to add @VonTeacher! :tada:

[kasugaijin]

@VonTeacher let me know if you have any questions!

[kasugaijin]

@VonTeacher hey are you still able to finish this? Please let me know in the next couple of days. I usually re-assign issues that sit without communication for a week or more. Thanks :)

[VonTeacher]

Thanks for the reminder and apologies for the delay. I'll spend some time ~this weekend~ now.

Edit: I updated the test file, taking into account your comments on the script and resolving conflicts that occurred as a result of #618. Awaiting workflow approval and any other comments you might have had on the home#index method.

[kasugaijin]

@VonTeacher No worries at all! Just making sure you have what you need and still able to work on it.

[kasugaijin]

@VonTeacher let me know if you can update re my feedback above. If not, I can also finish this off if you're busy. Let me know :)

[maebeale]

@kasugaijin (and @VonTeacher) might we want to merge this PR this weekend?

[kasugaijin]

@maebeale yes we can it just needs the tests removing or updating. I personally don’t think m this needs to be tested.

[meg-gutshall]

Resolved by #784