-
-
Notifications
You must be signed in to change notification settings - Fork 906
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
Missing listen feature added #668
Missing listen feature added #668
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we have to add audio files for each state in this case? Is there no other way?
Also, trvais checks are failing
@@ -0,0 +1,9 @@ | |||
<?xml version="1.0" encoding="UTF-8"?> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove .idea folder from commit
@@ -13,6 +14,8 @@ android: | |||
- extra-google-m2repository | |||
licenses: | |||
- '.+' | |||
before_install: | |||
- chmod +x gradlew |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are these changes required? If so, why?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Travis was failing so I referred to a post and made changes to solve but still the problem was nort solved.
private AudioManager mAudioManager; | ||
|
||
/** | ||
* This listener gets triggered when the {@link MediaPlayer} has completed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add javadoc style comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok I will do that
I tried with removing the whole functionality but that did not work so I approached in this way. |
Description
The audio clip was not playing while the data was fetched from the wiki pages. Now the audio files are added that will play the audio file when the user opts for the pronunciation of the word by clicking on
listen
Fixes #569
Type of change
Just put an x in the [] which are valid.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
./gradlew assembleDebug assembleRelease
./gradlew checkstyle
Checklist: