Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

add new blogs w/img #673

Merged
merged 10 commits into from
Jun 5, 2018
Merged

add new blogs w/img #673

merged 10 commits into from
Jun 5, 2018

Conversation

ValerieThoma
Copy link
Contributor

created new branch, old branch has package-lock.json conflicts

img for e2e with Selenium and Cypress
e2e with Selenium and Cypress
testing react apps with cypress
testing react apps with cypress
start your end to end testing with angular and cypress
start end to end testing with angular and cypress
sourceUrl: https://offering.solutions/blog/
date: May 27, 2018
author: Fabian Gosebrink
img: /img/examples/e2e-with-angular-and-cypress.jgp
Copy link
Member

Choose a reason for hiding this comment

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

Did you run this locally to ensure all of the images work correctly? I notice this has a .jgp extension, which does not look correct. You'll need to always run it locally and ensure the images and links are working.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch. Funny thing is, I did and always do run it locally. And the incorrect file extension did work. 🤷🏻‍♀️
screen shot 2018-06-05 at 10 31 44 am
screen shot 2018-06-05 at 10 31 19 am

Will fix.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Closer inspection reveals that the image extension is misspelled as well. So I guess that's why it worked. D'oh.

Copy link
Member

Choose a reason for hiding this comment

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

🙃

@jennifer-shehane jennifer-shehane merged commit 81aa1a5 into develop Jun 5, 2018
@jennifer-shehane jennifer-shehane mentioned this pull request Jun 7, 2018
brian-mann pushed a commit that referenced this pull request Jun 7, 2018
* add new blogs w/img (#673)

* add blog img

img for e2e with Selenium and Cypress

* add new blog

e2e with Selenium and Cypress

* add new blog

testing react apps with cypress

* add blog img

testing react apps with cypress

* add blog img

start your end to end testing with angular and cypress

* add new blog

start end to end testing with angular and cypress

* delete image with incorrect file extension

* new blog img

replaced image with .jpg file extension

* rename file extension 

change .jgp to .jpg

* Add PR template [skip ci]

* add html example to .not assertion (#678)

* update ci docs (#679)

* chore(deps): Update sinon to 5.1.0 🌟 (#680)

* chore(deps): update nock to 9.3.1 🌟 (#683)

* add quotes around --spec arg (#686)

* update ci docs
jennifer-shehane added a commit that referenced this pull request Jun 8, 2018
* add new blogs w/img (#673)

* add blog img

img for e2e with Selenium and Cypress

* add new blog

e2e with Selenium and Cypress

* add new blog

testing react apps with cypress

* add blog img

testing react apps with cypress

* add blog img

start your end to end testing with angular and cypress

* add new blog

start end to end testing with angular and cypress

* delete image with incorrect file extension

* new blog img

replaced image with .jpg file extension

* rename file extension 

change .jgp to .jpg

* Add PR template [skip ci]

* add html example to .not assertion (#678)

* update ci docs (#679)

* chore(deps): Update sinon to 5.1.0 🌟 (#680)

* chore(deps): update nock to 9.3.1 🌟 (#683)

* add quotes around --spec arg (#686)

* update ci docs

* chore(deps): update nock to 9.3.2 🌟 (#684)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants