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

Release/1.19.2 #1643

Merged
merged 70 commits into from
Aug 25, 2024
Merged

Release/1.19.2 #1643

merged 70 commits into from
Aug 25, 2024

Conversation

ThomasSession
Copy link
Collaborator

Merging 1.19.2

ThomasSession and others added 30 commits July 23, 2024 18:28
[SES-2441] Fix longpress message fails
…intent-permission

Adding back the fullscreen intent permission
* Fixed onion request crashes

* Use kotlin.Result instead

---------

Co-authored-by: fanchao <[email protected]>
bemusementpark and others added 28 commits August 5, 2024 20:15
Optimise SnodeAPI and Add Snode.Version and tests
# Conflicts:
#	app/build.gradle
#	app/src/main/java/org/thoughtcrime/securesms/calls/WebRtcCallActivity.kt
#	app/src/main/java/org/thoughtcrime/securesms/components/AvatarImageView.java
#	gradle.properties
* Removing unrequired APi checks

* Bumping min sdk to 26 and further cleaning version checks

* More clean ups

* Removed unused class
* Removing unrequired APi checks

* Bumping min sdk to 26 and further cleaning version checks

* More clean ups

* Removed unused class

* Clean up

* Clean up

* Clean up

* Clean up

* Clean up

* Clean up

* Clean up

* Clean up
* [SES-2551] capitalising sentences in convos

* [SES-2555] Add bg to image loading

* Using Robolectric to fix the tests
We need to rethink the UI tests if we do indeed want to use them as they are not tied to the existing flow anymore
* Replaced MLKit with ZXing for QR code scanning

* Adjusted some comment spacing

* Adjusted some comment phrasing

* Renamed MaybeScanQrCode to QRScannerScreen & removed double-import of ZXing core + removed ZXing android-integration

---------

Co-authored-by: alansley <[email protected]>
Co-authored-by: fanchao <[email protected]>
* Avoiding to recall Ipv4Int on every loop

* Avoiding memory allocation since this is used in laarge sets

* Further improvements and updated the memory allocation avoiding code

* PR feedback
Copy link
Collaborator

@AL-Session AL-Session left a comment

Choose a reason for hiding this comment

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

LGTM - but that being said, this is such a large PR I think we have to trust in the separate PRs comprising this to be confident in it.

@ThomasSession ThomasSession merged commit fd0f9ea into master Aug 25, 2024
2 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.

4 participants