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

[D000623] Rep. Mark DeSaulnier #1370

Closed
rshorey opened this issue Jan 5, 2015 · 11 comments
Closed

[D000623] Rep. Mark DeSaulnier #1370

rshorey opened this issue Jan 5, 2015 · 11 comments

Comments

@rshorey
Copy link
Member

rshorey commented Jan 5, 2015

Newly elected to 114th congress

@j-ro
Copy link
Contributor

j-ro commented Jan 14, 2015

tested and working

@j-ro j-ro closed this as completed Jan 14, 2015
@Hainish
Copy link
Contributor

Hainish commented Feb 4, 2015

This isn't working with poltergeist for me, but is working with watir. Are you encountering this as well @j-ro?

@j-ro
Copy link
Contributor

j-ro commented Feb 5, 2015

yep, needs watir. There's a bunch like that -- anywhere you see in the history me removing the double click, that needs watir.

See EFForg/phantom-of-the-capitol#54 -- I'd like to add a flag to each yaml, but we'd have to decide on congress-forms as a reference system re: the discussion here: #1402

@Hainish
Copy link
Contributor

Hainish commented Feb 5, 2015

I'd rather have no forms require watir. The reason is that watir is a huge memory hog. I'm trying to get to the bottom of why exactly it's messing up with phantomjs

@j-ro
Copy link
Contributor

j-ro commented Feb 5, 2015

So, a double click on the submit button will cause this one to work. I really don't know why, but that's the way I got them to work without watir. However, that messes up Sunlight's non-congress-forms implementation, so it was removed in favor of watir.

But if you find another way, great! I tried a ton of things, waiting, finding, etc... but couldn't get it to work.

@Hainish
Copy link
Contributor

Hainish commented Feb 5, 2015

I think if anything, there's no reason why we can't have a 'wedge' repo that describes which driver should be used for different members. I'm not sure about including it in the YAML, I think we should keep the YAML files as agnostic as possible to the implementation. But some really small repo like congress-forms-preferred-drivers could be used, to separate out the congress-forms hard logic of filling forms from the nitty-gritty of what works where.

@j-ro
Copy link
Contributor

j-ro commented Feb 5, 2015

I guess the issue is, because the implementations are so different, I don't feel being agnostic is really possible. When do we mark forms as passing? When one person's system works? Most of sunlight's recent commits need fixes for it to work with congress-forms, for example.

So yeah, a small public repo would help so we could quickly see if the form is passing with watir, but there are larger issues at play.

Anyway, see https://github.com/unitedstates/contact-congress/commits/01038bee9487a7cba5e69960db6579fa0ad365b5/members/E000294.yaml for what I've tried to get these to work before resorting to watir.

@Hainish
Copy link
Contributor

Hainish commented Feb 5, 2015

For this MoC I'm not having a problem of the form not submitting. I'm having a problem of it submitting, but being brought to a page which is not the "thank you" page that I see with watir. Pretty crazy stuff.

@j-ro
Copy link
Contributor

j-ro commented Feb 5, 2015

yep, there's a few like that, though less than the not submit. It's probably a related issue, since watir is the fix for all of them. I couldn't find anything in the HTML that was really different between ones that worked and ones that didn't. It was certain themes, so maybe there's something in the CSS or JS that's doing it, but I couldn't figure it out, and the screenshots weren't helpful.

@Hainish
Copy link
Contributor

Hainish commented Feb 5, 2015

works fine with capybara_webkit, added it to the array

@j-ro
Copy link
Contributor

j-ro commented Feb 5, 2015

oh, my mistake -- these are ones I use webkit on, not watir. Sorry about the confusion!

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

No branches or pull requests

3 participants