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

common NC android lib: identify common code to NC android files #1775

Closed
2 tasks done
mahibi opened this issue Jan 17, 2022 · 5 comments
Closed
2 tasks done

common NC android lib: identify common code to NC android files #1775

mahibi opened this issue Jan 17, 2022 · 5 comments
Labels
discussion Being discussed enhancement New feature or request

Comments

@mahibi
Copy link
Collaborator

mahibi commented Jan 17, 2022

For the future it is planned to extract code that is used both in https://github.com/nextcloud/android and https://github.com/nextcloud/talk-android (and maybe more NC android apps).

For the common code there should be a new nextcloud android lib.
Or maybe https://github.com/nextcloud/android-library is extended?
Also take into account https://github.com/stefan-niedermann/nextcloud-commons

This issue should be used to collect which code could be extracted to this common lib. Add ideas for common code to the list.
fyi @timkrueger @tobiasKaminsky @AlvaroBrey

Ideas for code which could be extracted:

todos for a common lib

  • new lib or existing lib?
  • which technologies/frameworks?
@tobiasKaminsky
Copy link
Member

We should also check if not needed packages are stripped out, e.g. if we include nextcloud-commons with e.g. 200 classes, but we use only 2, then proguard/other shrinking systems should remove the others.

@mahibi mahibi moved this to 🧭 Planning evaluation (don't pick) in 💬 Talk team Jan 24, 2022
@tobiasKaminsky
Copy link
Member

Plan

  • Talk: integrate Log_OC to test
  • Files: move LogsActivity to library
  • Talk: make use of LogsActivity

@tobiasKaminsky
Copy link
Member

Shrinking app:

@tobiasKaminsky
Copy link
Member

Update:

@mahibi
Copy link
Collaborator Author

mahibi commented Jan 30, 2023

i will close this issue.

we now have:

@mahibi mahibi closed this as completed Jan 30, 2023
@github-project-automation github-project-automation bot moved this from 🧭 Planning evaluation (don't pick) to ☑️ Done in 💬 Talk team Jan 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion Being discussed enhancement New feature or request
Projects
Archived in project
Development

No branches or pull requests

2 participants