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

[Doc] Update outdated ecommerce demo video #10236

Merged
merged 5 commits into from
Sep 24, 2024

Conversation

erwanMarmelab
Copy link
Contributor

Problem

Vimeo will delete our e-commerce video.

Solution

Change publication platform for YouTube

README.md Outdated Show resolved Hide resolved
docs/Readme.md Outdated Show resolved Hide resolved
packages/react-admin/README.md Outdated Show resolved Hide resolved
Copy link
Member

@fzaninotto fzaninotto left a comment

Choose a reason for hiding this comment

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

Missing replacements:

  • data generator readme
  • demo example readme
    And many more (look for 'vimeo' in the entire code)

README.md Outdated Show resolved Hide resolved
docs/Demos.md Outdated
@@ -208,7 +208,9 @@ If you want to see what react-admin is capable of, or if you want to learn from

The admin of a fictional poster shop, allowing to manage sales, products, customers and reviews. Built by the core team.

<div style="padding:56.25% 0 0 0;position:relative;"><iframe src="https://player.vimeo.com/video/474999017?h=4557e43b02&title=0&byline=0&portrait=0" style="position:absolute;top:0;left:0;width:100%;height:100%;" frameborder="0" allow="autoplay; fullscreen; picture-in-picture" allowfullscreen></iframe></div>
<div style="padding:56.25% 0 0 0;position:relative;">
<iframe width="1264" height="711" src="https://www.youtube.com/embed/gW6BvodBIzE" title="react admin e commerce demo" style="position:absolute;top:0;left:0;width:100%;height:100%;" frameborder="0" allow="accelerometer; autoplay; clipboard-write; encrypted-media; gyroscope; picture-in-picture; web-share" allowfullscreen></iframe>
Copy link
Member

Choose a reason for hiding this comment

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

use the nocookie domain (see other youtube embeds)

Co-authored-by: Francois Zaninotto <francois@marmelab.com>
@erwanMarmelab
Copy link
Contributor Author

erwanMarmelab commented Sep 24, 2024

Missing replacements:

  • data generator readme
  • demo example readme
    And many more (look for 'vimeo' in the entire code)

I don't agree. Those Vimeo videos refers to other videos and not this one (cf all videos we have)

@fzaninotto
Copy link
Member

I don't agree. Those Vimeo videos refers to other videos and not this one (cf all videos we have)

You're right, but the problem is even worse: the other video is a demo of react-admin v2, so it's even more outdated!

@fzaninotto fzaninotto merged commit 0a9de25 into master Sep 24, 2024
14 checks passed
@fzaninotto fzaninotto deleted the doc/change-vimeo-video-for-youtube branch September 24, 2024 16:33
@fzaninotto fzaninotto changed the title [Doc] Change Vimeo video for YouTube [Doc] Update outdated ecommerce demo video Sep 24, 2024
@fzaninotto fzaninotto added this to the 5.2.2 milestone Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFR Ready For Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants