Skip to content

📝 add section videos#883

Merged
akaszynski merged 7 commits intopyvista:masterfrom
tkoyama010:patch-7
Sep 7, 2020
Merged

📝 add section videos#883
akaszynski merged 7 commits intopyvista:masterfrom
tkoyama010:patch-7

Conversation

@tkoyama010
Copy link
Copy Markdown
Member

@tkoyama010 tkoyama010 commented Sep 4, 2020

Overview

📝 add section videos

Details

  • I created a section where we can share videos and materials when talking or doing tutorials at conferences about pyvista.
  • I am referring to the mayavi's this page.

ReadTheDocs 👈 check RTD html build on this branch

@tkoyama010 tkoyama010 changed the title 📝 add section videos 📝 add section videos Sep 4, 2020
tkoyama010 added a commit to pyvista/pyvista-doc-translations that referenced this pull request Sep 4, 2020
@tkoyama010
Copy link
Copy Markdown
Member Author

tkoyama010 commented Sep 4, 2020

ReadTheDocs 👈 check RTD html build on this branch

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 4, 2020

Codecov Report

❗ No coverage uploaded for pull request base (master@9c43de3). Click here to learn what that means.
The diff coverage is n/a.

@tkoyama010 tkoyama010 marked this pull request as ready for review September 4, 2020 12:10
@tkoyama010
Copy link
Copy Markdown
Member Author

It is ready for review 🚀

Copy link
Copy Markdown
Contributor

@GuillaumeFavelier GuillaumeFavelier left a comment

Choose a reason for hiding this comment

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

LGTM, what about the other @pyvista/developers ?

@akaszynski
Copy link
Copy Markdown
Member

Watched the videos and they looked great! Thanks for the addition.

@akaszynski akaszynski merged commit f3afbab into pyvista:master Sep 7, 2020
@tkoyama010 tkoyama010 deleted the patch-7 branch September 8, 2020 01:53
@akaszynski akaszynski mentioned this pull request Sep 8, 2020
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