Skip to content

Implemented API 31+ vibration service code to replace depreciated code #3755

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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

Conversation

u7484052
Copy link

Added code that fixes the following issue:
#3702

I added retro-compatibility as well if needed.

@shai-almog
Copy link
Collaborator

Can you remove the extra files from the commit?
There are 12 project files that shouldn't be a part of the commit.

@shai-almog
Copy link
Collaborator

Also please fix the compilations errors seen in the CI run. I'm guessing there's a missing import and you should use numeric values instead of constants for backward compatibility with older compilers.

@u7484052
Copy link
Author

Yeah sorry, I have fixed the importing error and removed the unnecessary files from the PR

@shai-almog
Copy link
Collaborator

This is still failing. I'm guessing it requires a fix to the library in the classpath but I'm not sure if this is something you can do. If not I'll have to look into it.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants