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

feat: Scheduled sending #1619

Open
wants to merge 58 commits into
base: master
Choose a base branch
from
Open

feat: Scheduled sending #1619

wants to merge 58 commits into from

Conversation

lebojo
Copy link
Contributor

@lebojo lebojo commented Nov 19, 2024

This PR introduces the Scheduled Send feature to the kMail iOS app, allowing users to schedule emails to be sent at a later time.

Key Changes

  • Added UI components for scheduling email (date and time picker).
  • Integrated scheduling logic with the existing email sending flow.
  • Updated backend API calls to handle scheduled send data.
  • Added validation for scheduling times (e.g., no past dates).
  • Enhanced error handling and user feedback for scheduling failures.
  • Reschedule header for scheduled email not sent yet
  • Localised strings
Capture d’écran 2024-11-19 à 16 40 33

Copy link

github-actions bot commented Nov 19, 2024

Found 4 unused code occurences

Expand
Mail/Views/New Message/Scheduled panel/ScheduleFloatingPanelView.swift:23:1: warning: Imported module 'MailResources' is unused
Mail/Views/New Message/Scheduled panel/ScheduleFloatingPanelView.swift:28:36: warning: Property 'draftManager' is unused
MailCore/Cache/MailboxManager/MailboxManageable.swift:67:24: warning: Parameter 'drafts' is unused
MailCore/Cache/MailboxManager/MailboxManager+Draft.swift:161:24: warning: Parameter 'drafts' is unused

@lebojo lebojo force-pushed the feat-Scheduled-sending branch 4 times, most recently from 72a3c83 to 07b27f2 Compare November 20, 2024 09:19
Copy link

sonarcloud bot commented Nov 20, 2024

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.

1 participant