-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
[RNMobile] Audio block capability now enables/disables media upload's media sources #31966
Merged
+44
−30
Merged
Changes from 69 commits
Commits
Show all changes
72 commits
Select commit
Hold shift + click to select a range
0519567
Basics of Audio block working
etoledom 690224d
Add audio support to MediaUpload
etoledom b7965fd
Add handling of file uploads and replace
etoledom f2ccb18
WPMediaLibrary support for Audio block
etoledom 2a02b83
Avoid removing media info on error state
etoledom 276b07d
Linting
etoledom 8a8b550
Added an AUDIO file to the test requestMediaPickFromMediaLibrary func
jd-alexander 9e03e63
Fixed typo in ToolbarButton of Audio Block.
jd-alexander 349aa19
Removed auto help behavior present on web that's not used on mobile.
jd-alexander 04a39b4
Merge branch 'master' into rnmobile/audio-block-I-b
jd-alexander 32029e0
[Android] Wired the click of the Audio Media Library button.
jd-alexander a18b8ac
Added Audio media options for choosing audio file locally.
jd-alexander af6cb99
[RNMobile] Audio Block: Proper caption field (#27689)
ceyhun 193339f
Add alert prompt to insert from URL
ceyhun 4ab2988
Set entered URL in audio block attributes
ceyhun d9a6041
Merge branch 'master' into rnmobile/audio-block-I-b
jd-alexander 1a46994
Merge branch 'master' into rnmobile/audio-block-I-b
jd-alexander f09c955
Merge branch 'master' into rnmobile/audio-block-I-b
jd-alexander fff09be
initial integration with react native prompt.
jd-alexander d4209e8
updated prompt lib.
jd-alexander 9d61d49
Merge branch 'rnmobile/audio-block-I-b' into rnmobile/audio-block-IV-…
jd-alexander f0a1034
updated commit hash of prompt lib.
jd-alexander 1eb58d2
Updated package-lock.json.
jd-alexander 17dea27
Added RNPromptPackage to Glue Code.
jd-alexander c7154d3
Merge branch 'master' into rnmobile/audio-block-IV-from-url
jd-alexander 0f597f3
fixed merge conflicts between the edit.native.js files of the branches
jd-alexander e4dec03
added onSelectURL to the media-upload component
jd-alexander f77a861
added onSelectURL to the media-placeholder
jd-alexander cad8e8a
updated onSelectURL support with notice and URL validation
jd-alexander ef478bb
sync package-lock.json
jd-alexander ee9a536
Merge remote-tracking branch 'origin/master' into rnmobile/audio-bloc…
jd-alexander 6b6221f
updated package-lock.json
jd-alexander 377ceae
updated package-lock.json
jd-alexander 01242f5
Merge branch 'trunk' into rnmobile/audio-block-IV-from-url
jd-alexander 5911b09
updated react-native-prompt-android setup due to binary dep changes.
jd-alexander cebdf5f
sync package-lock.json
jd-alexander 2dd3b6c
Integrated RNPromptPackage with the glue code.
jd-alexander f348df3
sync package-lock.json
jd-alexander b81dd77
package-lock.json fix
jd-alexander d4ad088
sync package-lock.json
jd-alexander 754b999
Added the globe icon to the Insert from URL source.Only shown on Android
jd-alexander e7ab21b
Extract filename from all URL types and optimized extension logic.
jd-alexander 1834b58
Merge branch 'trunk' into rnmobile/audio-block-IV-from-url
jd-alexander 614d57b
Merge branch 'trunk' into rnmobile/audio-block-IV-from-url
jd-alexander 6170600
Merge branch 'trunk' into rnmobile/audio-block-IV-from-url
jd-alexander 5acc1f2
Removed hook that would show/hide the audio block. It is not needed.
jd-alexander b5dac20
Only show the Insert from URL option when the audio block cap is false
jd-alexander 4d5b001
Added changelog entry for the Audio block URL prompt on free sites
jd-alexander 51494e3
Removed exclamation mark.
jd-alexander 1e550e6
Merge branch 'trunk' into rnmobile/audio-block-IV-from-url
jd-alexander e68550e
Reduced onPress -> onSelectURL logic since the parameters match.
jd-alexander 024ecad
Added entry to CHANGELOG.md
jd-alexander 2d1414b
Merge branch 'rnmobile/audio-block-IV-from-url' into rnmobile/add/aud…
jd-alexander 0e8db0f
simplified isAudioBlockEnabled's usage of getSettings.
jd-alexander ac6565a
exported the Insert from URL option so that the test can be utilize it.
jd-alexander ace7cc5
Added test that verifies the Insert From URL option with the audio type
jd-alexander 0153578
isAudioBlockMediaUploadEnabled now represents the Audio block capability
jd-alexander cec0331
placed the audio block capability in a single filter callback function
jd-alexander 6dec79a
Merge branch 'trunk' into rnmobile/add/audio-block-capability
jd-alexander 8010194
Added release notes entry under unreleased.
jd-alexander 39f1bf9
Merge branch 'trunk' into rnmobile/add/audio-block-capability
jd-alexander 9cb564e
Merge branch 'trunk' into rnmobile/add/audio-block-capability
jd-alexander 245a895
added logic that verifies that the source being enabled supports audio
jd-alexander f4aa392
formatted code changes.
jd-alexander d9a5da1
Merge branch 'trunk' into rnmobile/add/audio-block-capability
jd-alexander e94abc3
Changed the order of the props to match the order throughout the file.
jd-alexander 7d0ad16
Fixed order of ios capabilities
jd-alexander 11a79e7
Fix indent issues on Gutenberg props
fluiddot d382a9b
Merge branch 'trunk' into rnmobile/add/audio-block-capability
fluiddot b40012e
Merge branch 'trunk' into rnmobile/add/audio-block-capability
fluiddot 8422815
Merge branch 'trunk' into rnmobile/add/audio-block-capability
fluiddot 1ca8ae2
Merge branch 'trunk' into rnmobile/add/audio-block-capability
fluiddot File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
mediaSources
being used throughout this component lives within the bridge. I opted not to put this there since inserting from the URL is not bridge-related.