Conversation
|
Wow, thank you so much for taking care of this. Let me know how I can support you |
|
|
||
| # The name of the executable created for the application. Change this to change | ||
| # the on-disk name of your application. | ||
| set(BINARY_NAME "piggybank") |
There was a problem hiding this comment.
Could the name be changed to Oinkoin?
There was a problem hiding this comment.
The package is called that even before creating this PR. After testing desktop platforms, I'll take it as a final task.
pubspec.yaml
Outdated
| emoji_picker_flutter: ^4.0.0-dev.2 | ||
| reorderable_grid: ^1.0.10 | ||
| local_auth: ^2.3.0 | ||
| sqflite_common_ffi: ^2.3.4+4 |
There was a problem hiding this comment.
Accordingly to the readme of the package, this goes to dev_dependencies. For making it work on android and iOS another library seem needed https://pub.dev/packages/sqlite3_flutter_libs
So I am not sure if it is needed to keep both sqflite_common_ffi and sqflite.
There was a problem hiding this comment.
I'll try to replace and test flutter_libs.
|
Is it possible to make a web versions as well with Nextcloud or local sync? |
Sync is not the purpose of this PR, despite being an interesting feature. |
|
Hi, I have added the support for Linux. I suspect Windows and MacOS can be added very quickly now - but I don't have any Windows or MacOS device so I can't try this out. |
Thanks for your help. I'm sorry I didn't complete the Linux task, but I'll gladly test the latest commits on Win & Mac. Edit: |
@emavgl Linux bundles, starting with AppImage, do not work due to some issues related to flutter_distributor (now FastForge), as it's not a stable release too. 
While the workflow generates the exec file with custom name ("oinkoin"), the AppRun file fetches the current project name ("piggybank"), which means the AppImage booting will fail for trying to trigger a non-existing file.
Renaming the project, which works, might be the only solution, though the namespace in app stores must be updated. |
|
I fear changing name will break somehow the name for the android apks, which will break many things. Do you mind splitting the PR into multiple ones? It would make it easy to test, for example, the change you made on Linux side, replacing flutter_distributor. |
|
I fix the issue with the linux binary name, now it points to "piggybank" but it is ok, since the display name and the package name will stay Oinkoin. It works on my computer at least :) I tried both .deb and AppImage. I can't manage to switch to fastforge instead. I have problem building the appImage as I get this error here do you have the same error? |
3 participants
I started implementing this app for desktop platforms.
Roadmap
Platforms
Extra tasks