Skip to content

Conversation

@ellbristow
Copy link

Migration to 18.0

Minor changes made to code for compatiblity (ir.cron fields no longer exist), and fixed unit tests

Installs correctly, all unit tests pass

@matteonext
Copy link

Hi @ellbristow, I would like to test this module. Could you please rebase it to reload the runboat? Thanks

@matteonext
Copy link

@ellbristow rebase?

@github-actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale label Oct 12, 2025
@theangryangel theangryangel force-pushed the 18.0-mig-mail_activity_reminder branch 3 times, most recently from 5988d00 to e60984b Compare November 7, 2025 10:22
@theangryangel
Copy link
Member

Apologies, this one fell by the wayside and we've just come across it whilst being asked to port this module to 19.0 😅 I've rebased this against the current 18.0 tip and squashed the MIG commits.

We have been running this in production for some time.

Copy link
Member

@StefanRijnhart StefanRijnhart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After the rebase, 'Files changed' tab looks weird. Looks like unrelated modules like mail_thread_create_nolog are added now.

@StefanRijnhart
Copy link
Member

/ocabot migration mail_activity_reminder

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Nov 21, 2025
@OCA-git-bot
Copy link
Contributor

There's no issue in this repo with the title 'Migration to version 18.0' and the milestone 18.0, so not possible to add the comment.

@lorenzomorandini
Copy link

Hello everyone, could this be merged? Seems like everything is ok?

@theangryangel
Copy link
Member

Looking at it there is something wrong with the rebase, it's bringing in other modules. I'm not 100% sure why or how myself or Elliott managed to screw that up.

I'm not able to sort this today, but I will pop something in my calendar for Friday.

@theangryangel theangryangel force-pushed the 18.0-mig-mail_activity_reminder branch 2 times, most recently from 0a4eb07 to 0520d0c Compare January 2, 2026 10:24
@theangryangel theangryangel force-pushed the 18.0-mig-mail_activity_reminder branch from 0520d0c to fb7336b Compare January 2, 2026 10:26
@theangryangel theangryangel force-pushed the 18.0-mig-mail_activity_reminder branch from 60920ef to ed9e4bf Compare January 2, 2026 10:33
@theangryangel
Copy link
Member

theangryangel commented Jan 2, 2026

Should be sorted. Apologies for the noise this morning. New year brain haze :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.