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

Update 2013-07-19-ohad-samet-fraud-online-payments-risk-management.md #41

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

Conversation

rserna2010
Copy link
Contributor

Adding youtube video

@matin
Copy link
Member

matin commented Jul 26, 2013

@rserna2010 why did you set the width to 500px instead of filling up the blog container area?

@rserna2010
Copy link
Contributor Author

I just took the same dimensions you used for the Jeff Jordan video.

On Fri, Jul 26, 2013 at 2:19 PM, Matin Tamizi [email protected]:

@rserna2010 https://github.com/rserna2010 why did you set the width to
500px instead of filling up the blog container area?


Reply to this email directly or view it on GitHubhttps://github.com//pull/41#issuecomment-21648870
.

714-417-2727
[email protected]

@matin
Copy link
Member

matin commented Jul 26, 2013

I quickly pasted that in to show that it works. You can set it to whatever
you think is best.

@steveklabnik
Copy link
Contributor

This can't be merged, do we still need this?

@mahmoudimus
Copy link
Contributor

Yes, can we please merge this?

@mahmoudimus
Copy link
Contributor

@rserna2010 can you fix the conflicts?

@matin
Copy link
Member

matin commented Apr 30, 2014

ping @rserna2010

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.

4 participants