Skip to content

Conversation

@inattee
Copy link
Contributor

@inattee inattee commented Nov 29, 2024

Please ensure that your pull request meets the following requirements - thanks!

  • Does not contain merge commits. Rebase instead.
  • Contains commits with descriptive titles.
  • New code is written in Kotlin whenever possible.
  • Follows our existing codestyle (gradlew spotlessCheck to check and gradlew spotlessApply to format your source code; will be checked by CI).
  • Does not break any unit tests (gradlew testDebugUnitTest; will be checked by CI).
  • Uses a descriptive title; don't put issue numbers in there.
  • Contains a reference to the issue that it fixes (e.g. Closes #XXX or Fixes #XXX) in the body text.
  • For cosmetic changes add one or multiple images, if possible.

Finally, please replace this template text with a description of the change and additional context if necessary.

Copy link
Collaborator

@cketti cketti left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍

@cketti
Copy link
Collaborator

cketti commented Nov 29, 2024

Do we want to keep the old tablet screenshots (in sevenInchScreenshots and tenInchScreenshots) or remove them?

@kewisch
Copy link
Member

kewisch commented Nov 29, 2024

I think we can remove them for consistency, it doesn't look like they are shown in f-droid.

@kewisch kewisch force-pushed the screen-shot-updates branch from 2ffff6b to 6f07a7f Compare November 29, 2024 14:21
@kewisch kewisch force-pushed the screen-shot-updates branch from 6f07a7f to 6f4f25e Compare November 29, 2024 14:23
@kewisch kewisch merged commit 48beea4 into main Nov 29, 2024
2 checks passed
@kewisch kewisch deleted the screen-shot-updates branch November 29, 2024 14:23
kewisch added a commit that referenced this pull request Nov 29, 2024
K-9 Mail screen shot update for f-droid
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.

5 participants