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

[RNMobile] Remove mediaFilesCollectionBlock initial prop #58140

Conversation

fluiddot
Copy link
Contributor

@fluiddot fluiddot commented Jan 23, 2024

Related PRs:

What?

Remove initial prop mediaFilesCollectionBlock.

Why?

The associated logic that used this prop will be removed from the host app.

How?

  • Remove the initial prop.

Testing Instructions

Follow instructions from wordpress-mobile/gutenberg-mobile#6568.

Testing Instructions for Keyboard

N/A

Screenshots or screencast

N/A

@fluiddot fluiddot added [Type] Task Issues or PRs that have been broken down into an individual action to take Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) labels Jan 23, 2024
@fluiddot fluiddot marked this pull request as ready for review January 25, 2024 14:53
@dcalhoun dcalhoun self-requested a review January 25, 2024 15:42
Copy link
Member

@dcalhoun dcalhoun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fluiddot fluiddot merged commit 9a049b6 into trunk Jan 25, 2024
57 checks passed
@fluiddot fluiddot deleted the rnmobile/update/remove-initial-prop-media-files-collection-block branch January 25, 2024 17:23
@github-actions github-actions bot added this to the Gutenberg 17.7 milestone Jan 25, 2024
fluiddot added a commit that referenced this pull request Jan 25, 2024
* Remove `mediaFilesCollectionBlock` initial prop

* Remove `enableMediaFilesCollectionBlocks` from Gutenberg demo app
fluiddot added a commit that referenced this pull request Jan 26, 2024
* Release script: Update react-native-editor version to 1.111.0

* Release script: Update CHANGELOG for version 1.111.0

* Release script: Update podfile

* [RNMobile] Video block: Fix logic for displaying empty state based on source presence (#58015)

* Avoid displaying Video block empty state when a source is present

* Add tests to cover empty state visibility logic

* Update `react-native-editor` changelog

* Remove unneeded `await` statements from Video block unit tests

* Update `react-native-editor` changelog

* Release script: Update react-native-editor version to 1.111.1

* Release script: Update CHANGELOG for version 1.111.1

* Release script: Update podfile

* RNMobile: Avoid crashes by ensuring RichText value exists prior to `toString` calls (#58088)

* [RNMobile] Remove `mediaFilesCollectionBlock` initial prop (#58140)

* Remove `mediaFilesCollectionBlock` initial prop

* Remove `enableMediaFilesCollectionBlocks` from Gutenberg demo app

* Release script: Update react-native-editor version to 1.111.2

* Release script: Update CHANGELOG for version 1.111.2

* Release script: Update podfile

---------

Co-authored-by: Siobhan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) [Type] Task Issues or PRs that have been broken down into an individual action to take
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants