Good morning! I would like to take this ticket.

You have taken it!

Question regarding the Bonus: Also allow customization of the message reminding partners to pick up the distributions

I am having trouble finding a specific reminder message that goes out to partners who need to pick up. Is there one that I am missing or is this suggesting that if distribution.delivery_method is pick_up we should add another line to distribution_mailer/reminder_email reminding partners of this? Thanks!

You know -- I can't find it either. Please ignore the bonus -- if I find it, I'll create a new issue.

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.

Automatically unassigned after 7 days of inactivity.

Is this issue still open? Checking on it since it has the stale label and an open PR. If so, I'm happy to work on it.

Also, the open PR seems like a duplicate one from the closed one which was using the wrong approach as mentioned in the other PR comment, so I guess that can be closed 🤔

I was just noting that I need to check into this -- I'm pretty sure the functionality doesn't exist in production yet. Do you want to just start fresh?

@Sukhpreet-s If you want to start fresh, yes , we will close that PR.

Now that you asked me 😅 -- I took a thorough look at the PR and I think the existing code can be used. I'll try to use that PR for now. Thanks!

@cielf I realized that the old open PR (authored by @nathangthomas) is not merging in rubyforgood/human-essentials repo instead it's merging in the fork itself. And because the PR descriptions mentions this issue, hence the PR is getting tracked here even though it's not merging to this repo.

So I ended up creating a fresh PR.

@nathangthomas Can you please close your PR? Also I wonder if you accidentally created that PR with base branch as your own fork? 🤔