Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE] Add button to delete all mails and answers #660

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cweiske
Copy link
Contributor

@cweiske cweiske commented Mar 19, 2021

The "Mail list" page has a "cleanup" button that opens a small form
which allows one to delete all mails and answers on the current page.

Port of the old patch from 2018
#304

The "Mail list" page has a "cleanup" button that opens a small form
which allows one to delete all mails and answers on the current page.

Port of the old patch from 2018
 in2code-de#304
@einpraegsam einpraegsam force-pushed the develop branch 4 times, most recently from a31ca4e to 43b5957 Compare July 8, 2022 10:50
@mschwemer mschwemer deleted the branch in2code-de:master December 28, 2023 10:19
@mschwemer mschwemer closed this Dec 28, 2023
@mschwemer mschwemer reopened this Dec 28, 2023
@mschwemer mschwemer changed the base branch from develop to master October 2, 2024 15:48
@hannesbochmann
Copy link

This would be a great feature. @cweiske The PR needs some refactoring to resolve the merge conflicts.

@hannesbochmann
Copy link

@cweiske I think those things need to be changed to make it work again:

  • the translations are moved from the extension to the localization service of TYPO3. So the additions to de.locallang.xlf need to be added in another way. But I guess this is not that crucial.
  • the ModuleController::deleteAllMailsBeAction() should return a ResponseInterface with "return new ForwardResponse('list');"
  • in Resources/Private/Partials/Module/DeleteAllMails.html the form action should be "dispatch" and not "deleteAllMailsBe" and the form needs the element '<f:form.hidden name="forwardToAction" value="deleteAllMailsBe" id="forwardToAction" />'

@cweiske
Copy link
Contributor Author

cweiske commented Oct 24, 2024

The original patch was 2018, the new one 2021. It's 6 years later now and I currently don't have a project that uses powermail. I won't update the patch in the forseeable future.

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