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

Androidx (Replaced Butterknife with View Binding and refactored some code) with Android Studio 4.0 #111

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Skulltulla
Copy link

  • Copied project openvidu-android to openvidu-androidx
  • Replaced Butterknife with viewBinding build feature
  • Refactored SessionActivity
  • Updated manifest by moving all uses-permissions elements before application and added fullBackupContent to application per code suggestion
  • Added OpenViduUtils with method getBasicAuthString
  • Renamed buttonPressed to onClickStartFinishCall

- Replaced Butterknife with viewBinding build feature
- Refactored SessionActivity
- Updated manifest by moving all uses-permissions elements before application and added fullBackupContent to application per code suggestion
-  Added OpenViduUtils with method getBasicAuthString
- Renamed buttonPressed to onClickStartFinishCall
@ SessionActivity
- Added SharedPreferences to save and load url
- Added pseudo login functionality

@ CustomHttpClient
- Added httpGet(String url, Callback callback)...

@ activity_main.xml
- Made linearlayout orientation horizontal
- Added a new linearlayout to contain server, secret, and new login
  button
- Removed RelativeLayout
- Made secret text type to password

@ strings.xml
- Rearranged strings so similar values are together, e.g. buttons,
  openvidu, etc
@CSantosM CSantosM force-pushed the master branch 2 times, most recently from a157936 to 793aa9d Compare July 1, 2024 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants