nowinandroid icon indicating copy to clipboard operation
nowinandroid copied to clipboard

[Bug]: [Documentation]: Most of the code is written under src/main/java although its Kotlin

Open khalid64927 opened this issue 3 years ago • 2 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues

Is there a StackOverflow question about this issue?

  • [X] I have searched StackOverflow

What happened?

Since this is Android Showcase project It would be good to keep all codebase (which is in Kotlin) in relevant folder name (src/main/kotlin , src/test/kotlin, src/androidTest/kotlin)

The legacy naming for modules which is by default from IntelliJ ( src/main/java ) seems obsolete.

Screenshot 2022-11-15 at 7 43 19 PM

Relevant logcat output

No response

Code of Conduct

  • [X] I agree to follow this project's Code of Conduct

khalid64927 avatar Nov 15 '22 11:11 khalid64927

If anyone doesn't take this, then I can assign this to myself and make some contributions.

sumanabhi avatar Nov 23 '22 22:11 sumanabhi

Feel free to take this - thanks.

dturner avatar Nov 24 '22 10:11 dturner

Is this issue still open for contribution? If yes, then I would like to work on it!

RivanParmar avatar Mar 06 '23 06:03 RivanParmar

Plz do

Sincerely,

Avinash Dindial

On Sun, Mar 5, 2023, 10:43 PM Rivan Parmar @.***> wrote:

Is this issue still open for contribution? If yes, then I would like to work on it!

— Reply to this email directly, view it on GitHub https://github.com/android/nowinandroid/issues/436#issuecomment-1455540754, or unsubscribe https://github.com/notifications/unsubscribe-auth/AY75QEEMWGQFWD7JDES5DZDW2WBPNANCNFSM6AAAAAASA3D5TI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

Avinashdindial avatar Mar 06 '23 07:03 Avinashdindial

This has been fixed. image

dturner avatar Nov 06 '23 10:11 dturner