Skip to content

Conversation

@jguegel
Copy link
Contributor

@jguegel jguegel commented Dec 8, 2025

Type of change(s)

  • Bug fix
  • Feature / enhancement
  • Infrastructure / tooling (CI, build, deps, tests)
  • Documentation

What changed and why

  • Fixed new conversation shortcut behaving like thirdPartyIntent

Before & after preview

Old Behavior:
ignoreImageMinify
New Behavior:
ignoreImageMinify

Closes the following issue(s)

Checklist

  • I read the contribution guidelines.
  • I manually tested my changes on device/emulator (if applicable).
  • I updated the "Unreleased" section in CHANGELOG.md (if applicable).
  • I have self-reviewed my pull request (no typos, formatting errors, etc.).
  • All checks are passing.

@jguegel jguegel requested a review from naveensingh as a code owner December 8, 2025 15:50
@naveensingh naveensingh changed the title Fix New Conversation Shortcut 416 fix: properly handle "New conversation" shortcut Dec 8, 2025
Copy link
Member

@naveensingh naveensingh left a comment

Choose a reason for hiding this comment

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

There are two issues:

  • Intents such as sms:+19876543210 don't work anymore. Generally, it's a good idea to consider any potential side effects when making a change.
  • You stole my screenshots.

@jguegel
Copy link
Contributor Author

jguegel commented Dec 8, 2025

thanks for the input.
my own screenshot, i was honestly to lazy to blur them
ignoreImageMinify
ignoreImageMinify

fixed also the intent and tested with:
https://enzak.com/how-to-create-a-link-that-sends-an-sms-text-message/

Copy link
Member

@naveensingh naveensingh 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 now, thanks.

my own screenshot, i was honestly to lazy to blur them

I was messing with you.

@naveensingh naveensingh merged commit 988b532 into FossifyOrg:main Dec 9, 2025
6 checks passed
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.

New conversation shortcut doesn't work

2 participants