Project Plan

[djbrown]

Establish CI/CD

• Create CI-Build with GitHub Actions #6
• Create Nightly Build with GitHub Actions #7
• merge nightly-build into ci-build #15

Upgrade project setup to Android API Level 33

• Upgrade project setup to Android API Level 33 #17

Fix instrumentation tests

• Timeout waiting for App to become idle in OwnCloudExportTest #16
• Failing Instrumentation Test: PieChartReportTest.testSelectingValue #13
• Failing Instrumentation Test: TransactionsActivityTest#testAddMultiCurrencyTransaction #26

Migrate models to Kotlin

• codinguser@5dacb7d
• Converting models to Kotlin. #28
• Convert models to kotlin part2 #29

Changes from @JeanGarf #5

• #876 Major contribution to gnucash-android codinguser/gnucash-android#882
• #110 searchable spinner arrayadapter codinguser/gnucash-android#895
• ...

Changes from @XilinJia (Diff)

• ...

migrate translations

• Migration crowdin to weblate #3

Rename Project and Support Channels

• Update README links and references #1
• future support channel: GitHub issues
• also consider mailing lists, UserVoice ...

Rename Technical Name/ID in Source Packages etc.

• Migration to gnucash-pocket #2

[djbrown]

@JeanGarf @XilinJia do you agree to this plan, or do you think we should reorder based on some arguments?

[XilinJia]

I checked on @JeanGarf 's branch briefly. There is basically no change in models. And actually when migrating to Kotlin, preserving the models was quite some work.

Perhaps we can start merging @JeanGarf 's branch with this commit of mine (models code converted to Kotlin): codinguser@5dacb7d

This commit includes updates onto Androidx and only models codes converted to kotlin with the rest still in Java. The java codes include changes only related to calling getters and setters in the models (aside from any changes required for Androidx).

[djbrown]

Nice :) thanks for your feedback. Then we should switch the order entirely or just prepone this specific commit?
latter one would mean:

• merge codinguser@5dacb7d into master
• merge JeanGarf into master
• merge rest of XilinJia into master

what do you think about CI (using GitHub Actions), do you agree that we should establish that first?
Here is an example/tutorial: https://www.youtube.com/watch?v=4nVRJ9ulKJQ
We should modify that for all setup changes of course, but I think a successful central build is essential before doing code work.

I am very familiar with GitHub Actions, Gradle and Java, but not so much with Android and Kotlin.
So I think I'm able to create a first rudimentary pipeline. You could then create PRs which would be tested etc. before merging into master, so we can be sure not to break existing things centrally and transparently.

[XilinJia]

That order sounds good to me.
I think it would be good to set up CI first so to ensure each merge step problem free.

[JeanGarf]

I agree with your proposal.

Great !

[JeanGarf]

I checked on @JeanGarf 's branch briefly. There is basically no change in models. And actually when migrating to Kotlin, preserving the models was quite some work.

Perhaps we can start merging @JeanGarf 's branch with this commit of mine (models code converted to Kotlin): codinguser@5dacb7d

This commit includes updates onto Androidx and only models codes converted to kotlin with the rest still in Java. The java codes include changes only related to calling getters and setters in the models (aside from any changes required for Androidx).

I confirm, I didn't change the model. I just enhanced UI experience and corrected some calculus.

[arozaqf]

@djbrown can you please change the color for the tags, black on red is not easily readable. thanks

[djbrown]

@djbrown can you please change the color for the tags, black on red is not easily readable. thanks

@arozaqf do you mean the color of priority/now and priority/high? which theme are you using?
for me it looks like this:

that color (#B60205) is one of the predefined ones in the labels edit menu.
I could switch to a more vibrant red for more contrast (like type/bug). or do you suggest something else?

[djbrown]

also I updated the initial thread post on the top, to reflect the current state and todos