Benjamin Couey
Benjamin Couey
I tried reproducing the failing tests with the seeds `64284`, `55734`, `23313`, and `54905` (there wasn't any specific intention behind picking these seeds) on both my local machine and the...
`rspec './spec/system/adjustment_system_spec.rb[1:5:2:1]'` `seed 45953` Run log [here](https://github.com/rubyforgood/human-essentials/actions/runs/14117305703/job/39550487703?pr=5097). However, I have had no success in replicating it. I have tried running the individual test, the full file, and the full suite...
May I work on this one?
> @Benjamin-Couey Please see the above 3 points. 1/ There was supposed to be a migration that converted old `reminder_day` into an equivalent `reminder_schedule` but clearly something is broken. I...
> @Benjamin-Couey Please see the above 3 points. @cielf Alright, the first two points should have been addressed in the most recent commits. 1/ Fixed a bug with the migration...
@cielf while updating stuff so an entered start date remains persistent across page loads, some weird behavior I hadn't anticipated came up that I wanted your opinion on. Because the...
> Well, I suppose we could save it only if the other date info is filled in. > > That would mean that for the people who don't currently have...
@cielf Alright, pushed some commits addressing things that came up in the last office hours. Namely: - The start date field now comes after the monthly frequency field and is...
@dorner regarding your comments. I think I get the idea and I'll start working on reworking what I have. There is one point I'd like some clarification. > Then all...
@dorner Alright, the most recent [commit](https://github.com/rubyforgood/human-essentials/pull/5190/commits/6f98f503feff7c7da423091069a9ed03fcc6c702) contains my first pass on the new service object you described. I still need to hook it up everywhere besides the `Organization` page and...