Skip to content

Conversation

@christer77
Copy link
Member

1. View in List message
trans-folder-list-message

2. View in open message
trans-folder-open-message

@christer77 christer77 force-pushed the adding-restore-button-in-trash-folder branch 3 times, most recently from 86f438c to 4e77137 Compare October 9, 2025 09:10
@christer77 christer77 marked this pull request as ready for review October 9, 2025 09:34
@christer77 christer77 requested a review from kroky October 9, 2025 09:34
@kroky
Copy link
Member

kroky commented Oct 9, 2025

Can you prepare a similar PR for Tiki-Cypht integration (in Tiki codebase) and add some automated tests for this new functionality?

fix: Change AOL SMTP port from 587 to 465
@christer77 christer77 force-pushed the adding-restore-button-in-trash-folder branch from 4e77137 to dd09731 Compare October 28, 2025 08:24
@kroky kroky marked this pull request as draft October 29, 2025 10:15
@kroky
Copy link
Member

kroky commented Oct 29, 2025

I've requested 2 essential items 3 weeks ago. Putting this in draft until they are solved. Will review after that.

@christer77
Copy link
Member Author

Can you prepare a similar PR for Tiki-Cypht integration (in Tiki codebase) and add some automated tests for this new functionality?

Sorry for this delay, by the way, the question I ask myself is: In principle, Cypht integrates well into the Tiki-Cypht codebase, so what exactly am I going to do in Tiki?

@christer77 christer77 marked this pull request as ready for review October 29, 2025 10:54
@kroky
Copy link
Member

kroky commented Oct 29, 2025

I just receive notifications to review and wasn't sure if there was anything new to renew.

Tiki has tracker storage support for emails. You can delete emails and move to trash folder inside a tracker item. Restore functionality should be added specifically there in the integration if we want to be consistent.

@christer77 christer77 marked this pull request as draft October 30, 2025 12:09
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.

3 participants