-
Notifications
You must be signed in to change notification settings - Fork 17
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
Report enhancements made by JeanGarf / gnucash-android #5
Comments
@JeanGarf do you think the project rename should wait until those major contributions are merged in? |
Does it fit XilinJia's commits? https://github.com/XilinJia/gnucash-android-kotlin I've read he made a lot od work recently, e.g. rewriting to Kotlin. |
I know XilinJia is claimed to be the GnuCash-Pocket developer, but he never asked for it and didn't say a word about that after he "got the job" – so maybe my comment above is pointless… 🤷 |
OK, I know I need to say hi to everyone: hi! Answered to questions from @Kaligula0: To all, are we going to stay with Java, or get on with Kotlin? I'd be happy that we start from my Kotlin code base. |
I don't know the dis/advantages of moving to Kotlin. But, firsty, l I see that much of the work is done (thanks XilinJia!) and, secondly, I also commit to another Android app on GitHub where the developer recently moves constantly to Kotlin. And what I learned from quick googling – Kotlin got a language of choice when it comes to Android, so I guess Kotlin is more future-proof and will be easier to maintain. 🤔 |
I everybody, I'm very pleased to see all these enthusiastics to restart the project ! I don't have lot of time at the moment and I don't know Kotlin, therefore It will be hard for me to contribute in the next months. However, I would be pleased to answer you about my previous developments if you have questions when merging, about functionalities I developed. See you... |
It would have been easier to merge before renaming, but now it is in Kotlin, so the merge will be difficult, renamed or not ! So now, I think it is not really the hardest problem ! |
@JeanGarf, can any of your contributions be merged to this repo in its current state? |
@Kaligula0 Yes, my most recent commit is here 535f3d5 (https://github.com/JeanGarf/gnucash-android/tree/tw_develop) and contains all my contributions. |
Expected behaviour
Integrate JeanGarf's Pull Requests made in https://github.com/codinguser/gnucash-android/pulls, particularly :
Software specifications
The text was updated successfully, but these errors were encountered: