Skip to content
This repository was archived by the owner on Jan 15, 2020. It is now read-only.

add demo video #49

Closed
wants to merge 1 commit into from
Closed

add demo video #49

wants to merge 1 commit into from

Conversation

dhmlau
Copy link
Member

@dhmlau dhmlau commented Oct 19, 2018

Signed-off-by: Diana Lau dhmlau@ca.ibm.com

Add demo video to showcase how to create REST APIs with just 5 steps.
Eventually, I think we want to have a page for presentations and videos in the LB4 web site.

related: #47

Signed-off-by: Diana Lau <dhmlau@ca.ibm.com>
@dhmlau dhmlau self-assigned this Oct 19, 2018
@raymondfeng
Copy link
Member

IMO, github is not the best place to host videos. Should we upload it to YouTube and embed in v4.loopback.io site?

@bajtos
Copy link
Member

bajtos commented Oct 19, 2018

+1 for hosting videos outside of GitHub. At minimum, we should use LFS extension for git.

Can we use IBM infrastructure to host these videos? YouTube is very unfriendly to user privacy.

@dhmlau
Copy link
Member Author

dhmlau commented Oct 19, 2018

I've checked with the IBM Developer editor. They actually are using YouTube https://www.youtube.com/user/developerworks.
An alternative I see others are using is Vimeo.

@bajtos
Copy link
Member

bajtos commented Oct 22, 2018

I've checked with the IBM Developer editor. They actually are using YouTube https://www.youtube.com/user/developerworks.
An alternative I see others are using is Vimeo.

Fair enough 🤷‍♂️

@dhmlau
Copy link
Member Author

dhmlau commented Oct 24, 2018

Closing this PR as irrelevant.

  • Got feedback from @bajtos on the video
    • add narrative
    • more descriptive datasource name
    • look into the security vulnerability when running lb4 app, which @raymondfeng has a PR up already.
  • When it's ready, will post the video on YouTube instead in our github repo.

@dhmlau dhmlau closed this Oct 24, 2018
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants