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

Non-React view option for external-views #3122

Merged
merged 5 commits into from
Aug 11, 2020
Merged

Non-React view option for external-views #3122

merged 5 commits into from
Aug 11, 2020

Conversation

ChumpChief
Copy link
Contributor

Feedback on the tutorial was that it would be accessible to a broader audience if it didn't require React knowledge. Putting up this DOM-only draft PR for an alternative to consider.

Copy link
Contributor

@SamBroner SamBroner left a comment

Choose a reason for hiding this comment

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

This is great. I think this will be easier for people to approach actually... just javascript taking in some data.

Copy link
Contributor

@skylerjokiel skylerjokiel left a comment

Choose a reason for hiding this comment

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

:shipit:

@ChumpChief ChumpChief marked this pull request as ready for review August 11, 2020 17:10
@ChumpChief ChumpChief merged commit c851ee9 into microsoft:master Aug 11, 2020
@ChumpChief ChumpChief deleted the AppIntegrationIterate2 branch August 11, 2020 17:22
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