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 video link to readme #69

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ooker777
Copy link

No description provided.

@vigri
Copy link
Member

vigri commented Jul 25, 2023

Thank you very much for this PR.
I've to questions:

  1. Are you the author of this video? If yes, can me make this video "public"? When I open your link I'm forced to create a vimeo account to watch the video
  2. The uploaded video is 13 years old. What do you think, is this still "up to date" despite the age?

@vigri vigri added the documentation Improvements or additions to documentation label Jul 25, 2023
@vigri vigri self-assigned this Jul 25, 2023
@Piolie
Copy link

Piolie commented Jul 25, 2023

The author of this video is Joseph Artsimovich (tulon), the creator of the original Scan Tailor. I'm not sure if it should be added without at least a heads-up about its origin and age.

@ooker777
Copy link
Author

hmm can you check again? I don't need to login to Vimeo to watch it. So I guess it's public anyway.

I think as long as the UX is not much different, then we can reuse the videos?

@vigri
Copy link
Member

vigri commented Jul 26, 2023

hmm can you check again? I don't need to login to Vimeo to watch it. So I guess it's public anyway.

I think as long as the UX is not much different, then we can reuse the videos?

I've opened the video again:

image

@Piolie
Copy link

Piolie commented Jul 26, 2023

I can watch it without using an account. Maybe it's region blocked.
I guess it adds some value to the documentation, but it needs further explaining.

@mara004
Copy link

mara004 commented Aug 2, 2023

@ooker777 Thanks a lot for bringing this up, I did not know Tulon had made videos about ScanTailor. This is quite interesting.

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

Successfully merging this pull request may close these issues.

4 participants