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

decrease file size #73

Closed
wants to merge 1 commit into from
Closed

decrease file size #73

wants to merge 1 commit into from

Conversation

saaksshi
Copy link

File size for 2 min audio was 21 MB.
Now with lower bit rate it's been decreased to almost half

Platforms affected

iOS

What does this PR do?

File size for 2 min audio was 21 MB.
Now with lower bit rate it's been decreased to almost half

What testing has been done on this change?

Create/Play/pause audio

Checklist

  • Reported an issue in the JIRA database
  • Commit message follows the format: "CB-3232: (android) Fix bug with resolving file paths", where CB-xxxx is the JIRA ID & "android" is the platform affected.
  • Added automated test coverage as appropriate for this change.

@cordova-qa
Copy link

Cordova CI Build has completed successfully.

Commit - Link
Dashboard - Link

Builder Name Console Output Test Report Device Logs
Windows 8.1 Store Link Link Link
Windows 10 Store Link Link Link
Windows 8.1 Phone Link Link Link
iOS Link Link Link
Android Link Link Link

@ronelisaac
Copy link

Hi, can you help me with anything? I want you to take the video, do not preview it, save it at once, can you help me?

@saaksshi
Copy link
Author

saaksshi commented Mar 13, 2017 via email

@filmaj
Copy link
Contributor

filmaj commented Apr 25, 2017

I don't think we will be accepting this. Only a single-platform change, setting to an arbitrary bit rate. At a minimum, this would have to be a part of the spec across all platforms, with changes to all platforms.

Furthermore, this plugin is based on the media capture API: https://www.w3.org/TR/html-media-capture/

That should be the starting point for these kinds of changes :)

Will close this down.

@asfgit asfgit closed this in 0c9895e Apr 25, 2017
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.

None yet

4 participants