Skip to content

Conversation

@ymow
Copy link
Contributor

@ymow ymow commented Apr 10, 2016

Add PhoneCallReceiver to catch PhoneCall event( include incoming/outgoing call), start/stop video recording feature will following PhoneCall State. After recording is finished, send notification and play the video.

Because BroadcastReciver via PhoneCall need use start/stop
recording feature with Service,
So Service add start/stop methods

Update gitignore and remove untracking files

Ignore all files in .idea/ folder

Add callback to recieve recording result

Also show some Toast to notify the recording state.

Support start/stop recording feature

MyVideoList init
@chartsai
Copy link
Owner

I think the changes are too many... can you squash these changes to only 1 commit?
And, don't modify the current formatting (e.g. add a unnecessary space or empty line), It makes someone confused when tracing issue.

@ymow ymow force-pushed the PhoneCallReceiver_recording_and_playing6 branch from c4defd7 to 356bcd5 Compare April 11, 2016 11:54
merge start/stop recording feature during Incoming and
OutGoing PhoneCall State. PhoneCallReceiver will
trigger the ServiceCarema recording. After recording
is finished, send loca Push Notification.

add VideoPlayer after click local push notification

When phone call finish will let media recording` finsh
and send push notification, click the item will make
PhoneCall video play again.

issue:When ServiceCarema get onStartCommand probably crash
onStartCommand is execute by startToStartRecording,
means some condition make context of PhoneCall is Null.

Update gitignore and remove untracking files

Ignore all files in .idea/ folder

remote conflicts ff
@ymow ymow force-pushed the PhoneCallReceiver_recording_and_playing6 branch from 356bcd5 to 952b2d8 Compare April 11, 2016 14:08
@chartsai
Copy link
Owner

The first commit (create by me) is not needed, please clear your patch.

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