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

Add support for cathash image creation in android client #88

Conversation

HenrikJannsen
Copy link
Contributor

@HenrikJannsen HenrikJannsen commented Nov 30, 2024

Based on #86

We have to build the cathash image on the client as otherwise the loading of the images would take too long. We cache the images once created so they are only generated once.
Beside that to provide the image from the Rest API in headless mode would have been quite a hack (I got it working but did not like it - required to start JavaFX in headless mode via the monocle test framework which supports that).

I tried to add iOS support but did not get the images displayed. Beside that the iOS code is ChatGPT derived and must be reviewed/improved/fixed by a dev familiar with iOS.

@HenrikJannsen HenrikJannsen force-pushed the add-support-for-cathash-image-creation-in-android-client branch from 3bb16a3 to 2ceb8d3 Compare November 30, 2024 13:04
@HenrikJannsen HenrikJannsen marked this pull request as draft December 1, 2024 05:19
@HenrikJannsen
Copy link
Contributor Author

Continue working on ios.... got the loading of the images resolved. Still having problems to display it...

@HenrikJannsen HenrikJannsen force-pushed the add-support-for-cathash-image-creation-in-android-client branch from 1c315bd to e207890 Compare December 1, 2024 08:27
@HenrikJannsen HenrikJannsen marked this pull request as ready for review December 1, 2024 08:27
@rodvar
Copy link
Collaborator

rodvar commented Dec 2, 2024

henrik still working on this one, marking as draft

@rodvar rodvar marked this pull request as draft December 2, 2024 00:27
@HenrikJannsen HenrikJannsen marked this pull request as ready for review December 2, 2024 01:26
HenrikJannsen and others added 17 commits December 2, 2024 08:29
…e need the application base directory.

Pass context to AndroidApplicationService.
Fix package names.

Add image to generateKeyPair as type Any.
This is just preliminary due KMP issues with BitMap or related image formats.
 - Implemented service dependant on ImageUtils for android node
 - provided unit tests using Roboelectric emulated instrumentation
Add KMP compatible cathash implementation for clients.

This is java code ported from Bisq 2.

Move ImageUtil to shared
Add PlatformPainter
Move platform abstractions to one file per module
@HenrikJannsen HenrikJannsen force-pushed the add-support-for-cathash-image-creation-in-android-client branch from dcf03d6 to f9e8aba Compare December 2, 2024 01:39
HenrikJannsen and others added 4 commits December 2, 2024 08:48
 - provide serialziation helpers for it
 - fix usage of actual for android, and usage of ImageBitmap for the expected PlatformImage
…er saved profile. Added same code for iOS though not tested yet
@rodvar rodvar force-pushed the add-support-for-cathash-image-creation-in-android-client branch from 9dbab3a to ff1aace Compare December 3, 2024 22:39
@rodvar
Copy link
Collaborator

rodvar commented Dec 3, 2024

@HenrikJannsen Haven't been able to work this properly in the xClients so decided to make my way through it understanding the node part better. I integrate your work with the presentation architecture so now the final generated cathash is accessible to the rest of the app components via a repository.

It works like charm on androidNode, I've did all the changes considering android and iOS and fix some issues and compilation along the way so later today when I get back to the xClients hopefully the way with be paved to success :)

I'll let you know, I can smell the merge coming soon... please have a look at my changes if you are around 💪

@HenrikJannsen
Copy link
Contributor Author

Ah great that you could get it working on iOS!!! And added support for the top bar!

@rodvar rodvar merged commit f316a01 into bisq-network:main Dec 4, 2024
1 check passed
@HenrikJannsen HenrikJannsen deleted the add-support-for-cathash-image-creation-in-android-client branch December 12, 2024 13:51
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