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

Report enhancements made by JeanGarf / gnucash-android #5

Open
JeanGarf opened this issue Oct 5, 2022 · 10 comments
Open

Report enhancements made by JeanGarf / gnucash-android #5

JeanGarf opened this issue Oct 5, 2022 · 10 comments
Labels
enhancement New feature or request priority/high

Comments

@JeanGarf
Copy link

JeanGarf commented Oct 5, 2022

Expected behaviour

Integrate JeanGarf's Pull Requests made in https://github.com/codinguser/gnucash-android/pulls, particularly :

Software specifications

  • GnuCash Android version:
  • System Android version: 9
  • Device type: Galaxy Note 8
@djbrown
Copy link

djbrown commented Oct 5, 2022

@JeanGarf do you think the project rename should wait until those major contributions are merged in?

@Kaligula0
Copy link
Collaborator

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.

@Kaligula0
Copy link
Collaborator

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… 🤷

@XilinJia
Copy link

OK, I know I need to say hi to everyone: hi!

Answered to questions from @Kaligula0:
codinguser#913 (comment)

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.

@Kaligula0
Copy link
Collaborator

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. 🤔

@djbrown djbrown mentioned this issue Oct 11, 2022
19 tasks
@djbrown
Copy link

djbrown commented Oct 11, 2022

I think we should definitively migrate to Kotlin, especially since @XilinJia already did the work ❤️ .
But I'm not sure in which order we should merge the contributions. Let's discuss that in #8

@djbrown djbrown added enhancement New feature or request priority/high labels Oct 13, 2022
@djbrown djbrown added this to the First official release 🎉 milestone Oct 13, 2022
@JeanGarf
Copy link
Author

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.
I know I didn't do well about internationalization because I didn't know Crowdin, but all features work well on Samsung Galaxy Note 8.

See you...

@JeanGarf
Copy link
Author

@JeanGarf do you think the project rename should wait until those major contributions are merged in?

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 !

@Kaligula0
Copy link
Collaborator

@JeanGarf, can any of your contributions be merged to this repo in its current state?

@JeanGarf
Copy link
Author

@Kaligula0 Yes, my most recent commit is here 535f3d5 (https://github.com/JeanGarf/gnucash-android/tree/tw_develop) and contains all my contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority/high
Projects
None yet
Development

No branches or pull requests

4 participants