Skip to content

[Fix, Add] Fix New props import pattern, Added New components#118

Merged
pranshuchittora merged 1 commit intoreact-native-elements:masterfrom
khushal87:khushal87-dev1
Jun 22, 2021
Merged

[Fix, Add] Fix New props import pattern, Added New components#118
pranshuchittora merged 1 commit intoreact-native-elements:masterfrom
khushal87:khushal87-dev1

Conversation

@khushal87
Copy link
Member

@khushal87 khushal87 commented Jun 9, 2021

What kind of change does this PR introduce?

Did you add tests for your changes?
Not required
If relevant, did you update the documentation?
Yes
Summary
This PR fixes the prop import issue and adds new playground for new components added in the main package.

Does this PR introduce a breaking change?
Yes

Other information
Not required

@netlify
Copy link

netlify bot commented Jun 9, 2021

👷 Deploy Preview for rne-playground processing.

🔨 Explore the source changes: 2b16301

🔍 Inspect the deploy log: https://app.netlify.com/sites/rne-playground/deploys/60c054cac03c780007d7486e

);
exec("cp -r ./react-native-elements/website/docs/props ./src/content/Props"),
exec(
"cp -r ./react-native-elements/website/docs/main/props ./src/content/Props"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"cp -r ./react-native-elements/website/docs/main/props ./src/content/Props"
"cp -r ./react-native-elements/website/docs/props ./src/content/Props"

Copy link
Contributor

@jugshaurya jugshaurya Jun 20, 2021

Choose a reason for hiding this comment

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

@pranshuchittora this file has some substantial changes in #97 please look there and this thing isn't a problem there! Please look at the PRs available. those PRs have to be merged before the latest ones. Otherwise, there will be conflicts. @pranshuchittora

@pranshuchittora pranshuchittora merged commit 281416e into react-native-elements:master Jun 22, 2021
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