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

[Mobile] - Refactor BlockListItem #5660

Merged

Conversation

wpmobilebot
Copy link
Collaborator

Related PRs

Description

This PR is generated by version-toolkit to downstream the changes for gutenberg submodule.

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Apr 12, 2023

Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job!

@wpmobilebot
Copy link
Collaborator Author

This PR is closed because there is no longer an associated gutenberg PR for it.

If you'd like to keep a PR open after its upstream counterpart is closed, please assign it to a team member or create a new commit.

@wpmobilebot wpmobilebot deleted the version-toolkit/gutenberg/rnmobile/refactor-block-list-item branch April 13, 2023 16:33
@geriux geriux restored the version-toolkit/gutenberg/rnmobile/refactor-block-list-item branch April 13, 2023 16:35
@geriux geriux reopened this Apr 13, 2023
@geriux geriux self-assigned this Apr 13, 2023
@geriux geriux force-pushed the version-toolkit/gutenberg/rnmobile/refactor-block-list-item branch from bef95ee to 8f2e87f Compare April 13, 2023 20:16
@geriux geriux changed the base branch from trunk to block-list-refactor April 13, 2023 20:18
@geriux geriux force-pushed the version-toolkit/gutenberg/rnmobile/refactor-block-list-item branch 3 times, most recently from 83a0275 to d8013a8 Compare April 26, 2023 10:04
@geriux geriux force-pushed the version-toolkit/gutenberg/rnmobile/refactor-block-list-item branch from d8013a8 to e1a9f38 Compare May 3, 2023 16:33
@geriux geriux changed the base branch from block-list-refactor to trunk May 3, 2023 16:33
@geriux geriux requested a review from dcalhoun May 3, 2023 16:33
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.

Approved via WordPress/gutenberg#49765 (review). Looks like we have a single visual test failing, we may need an updated snapshot.

Copy link
Member

Choose a reason for hiding this comment

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

This snapshot was updated slightly. It appears the sibling visual test for Android is failing. Maybe it needs an updated snapshot as well?

@geriux geriux merged commit c554b69 into trunk May 4, 2023
@geriux geriux deleted the version-toolkit/gutenberg/rnmobile/refactor-block-list-item branch May 4, 2023 13:06
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.

3 participants