Cory Streiff

Results 47 comments of Cory Streiff

The app we're working on isn't yet live, so I have yet to fill out the form. But I am curious about the information gathered. Will you share any anonymized...

Hey @felicia22! I think this task is outdated and could probably be closed. Looking at [this seed file](https://github.com/rubyforgood/pet-rescue/commit/07a641bba789559f03eeeb50bced07b6efee874f#diff-02d8f4d1d92e7df82f4c8891106567ac252dbe93758acba470918355efaa5c82) from a PR submitted right before this issue was opened, we can...

Looks like this task was completed with #918. Can this issue be closed?

I could take this issue on :raising_hand: Out of all the filters mentioned by @meg-gutshall, the only one that looks like it would be more effort to implement is the...

@kasugaijin Thanks for the loom, that really clears up what acceptance criteria for this task would be. Also agree with your suggestion to move this logic into a concern to...

@TheFaizanAnwar It looks like you've committed the database migration file, but not the changes that it made to `db/schema.rb`. During the workflow that runs the tests, migrations aren't run, the...

@cielf When I first saw this message of yours, a different contributor had already gotten assigned to this task. But it looks like no one is currently assigned so I...

I fixed the merged conflict and added the schema file changes, so the tests are now passing. So I think this is ready for review again.

@cielf I added the limit to 500 characters. Could you check it once again? I copied the implementation from elsewhere in the app, so you shouldn't be able to input...

There is an issue with the formatting of validation errors, but I think that is a separate issue: ![image](https://github.com/user-attachments/assets/482ed4d5-1067-46b8-80d5-8ec899b178b8)