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

add sample for adding WebPart with properties #1596

Closed
wants to merge 1 commit into from

Conversation

ypcode
Copy link
Contributor

@ypcode ypcode commented May 16, 2020

Closes #1438

@coveralls
Copy link

coveralls commented May 16, 2020

Coverage Status

Coverage remained the same at 100.0% when pulling 2836e73 on ypcode:issue/1438-webpart-properties into 141800a on pnp:dev.

@garrytrinder
Copy link
Member

garrytrinder commented May 17, 2020

Thanks @ypcode 👏🏻

@waldekmastykarz waldekmastykarz self-assigned this May 20, 2020
Copy link
Member

@waldekmastykarz waldekmastykarz left a comment

Nicely done with a few minor adjustment I've done when merging the PR 👏

@waldekmastykarz
Copy link
Member

waldekmastykarz commented May 20, 2020

Merged manually. Thank you! 👏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants