Skip to content

Improved RNTester deeplink support to go straight to a specific example#41981

Closed
fkgozali wants to merge 1 commit into
facebook:mainfrom
fkgozali:export-D52227013
Closed

Improved RNTester deeplink support to go straight to a specific example#41981
fkgozali wants to merge 1 commit into
facebook:mainfrom
fkgozali:export-D52227013

Conversation

@fkgozali
Copy link
Copy Markdown
Contributor

Summary:
Improved RNTester URL deeplink support to cover:

  • rntester://example/<moduleKey>
  • rntester://example/<moduleKey>/<exampleKey>

Extra details:

  • For example modules that do not specify showIndividualExamples: true, allow deeplink URL with the specific exampleKey to only render the specific example, instead of all of them.
  • Added flexibility for moduleKey: search for optional suffixes ("Index", "Example").
  • Adjusted Back button action to properly go back to the root after a deeplink.
  • Added example-container generic testID on the example wrapper component.

Changelog: [Internal]

Differential Revision: D52227013

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 18, 2023
@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D52227013

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D52227013

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D52227013

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D52227013

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D52227013

…le (facebook#41981)

Summary:

Improved RNTester URL deeplink support to cover:
*  `rntester://example/<moduleKey>`
*  `rntester://example/<moduleKey>/<exampleKey>`

Extra details:
* For example modules that do not specify `showIndividualExamples: true`, allow deeplink URL with the specific exampleKey to only render the specific example, instead of all of them.
* Added flexibility for moduleKey: search for optional suffixes ("Index", "Example").
* Adjusted Back button action to properly go back to the root after a deeplink.
* Added `example-container` generic testID on the example wrapper component.

Changelog: [Internal]

Reviewed By: yungsters, NickGerleman

Differential Revision: D52227013
fkgozali added a commit to fkgozali/react-native that referenced this pull request Dec 21, 2023
…le (facebook#41981)

Summary:

Improved RNTester URL deeplink support to cover:
*  `rntester://example/<moduleKey>`
*  `rntester://example/<moduleKey>/<exampleKey>`

Extra details:
* For example modules that do not specify `showIndividualExamples: true`, allow deeplink URL with the specific exampleKey to only render the specific example, instead of all of them.
* Added flexibility for moduleKey: search for optional suffixes ("Index", "Example").
* Adjusted Back button action to properly go back to the root after a deeplink.
* Added `example-container` generic testID on the example wrapper component.

Changelog: [Internal]

Reviewed By: yungsters, NickGerleman

Differential Revision: D52227013
@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D52227013

@github-actions
Copy link
Copy Markdown

This pull request was successfully merged by @fkgozali in a47d7c5.

When will my fix make it into a release? | Upcoming Releases

@github-actions github-actions Bot added the Merged This PR has been merged. label Dec 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants