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.

Sign up for GitHub

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

Jump to bottom

Fix Issue #1 - Changes to order of forms in Shopify's activate account page have broken mechanize_activate_customer_login.rb #2

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

Fix Issue #1 - Changes to order of forms in Shopify's activate account page have broken mechanize_activate_customer_login.rb #2

wants to merge 1 commit into from

Commits on Jul 27, 2012

  1. Fix Issue joelvh#1 - Changes to order of forms in Shopify's activate … 

    …account page have broken mechanize_activate_customer_login.rb

To try and add a bit of resilience to future undocumented changes made by Shopify, the script will now loop through all the forms on the page, and identify the form which contains the 'password' fields. This will then assign the relevant form to the activate_form variable.

While this solution is not completely bulletproof against future changes made by Shopify, it should mean that any future structural changes to that page will not break the mechanize script's functionality.
    @michaelrshannon
    michaelrshannon committed Jul 27, 2012
    Configuration menu
    Copy the full SHA
    a65b94b View commit details
    Browse the repository at this point in the history