Peter Woo

Results 16 comments of Peter Woo

The county field is free-form text in the actual contact form[1]. From looking at message-form.js/message-form.html, it seems democracy.io expects it always to be a ``. [1] https://www.tomudall.senate.gov/contact/email-tom

Thanks @randylubin! I've added notes on some things I think should be addressed before we merge (email config and error handling).

Thanks for taking a look! I fixed the syntax and style issues you mentioned, and restored the bypass behavior for pre-formatted blocks. I'd misinterpreted what that check was for and...

Some previous discussion: https://github.com/mattpardee/democracy.io/issues/38 https://github.com/mattpardee/democracy.io/issues/94 I like the idea of adding a ? tooltip next to the field.

Also we need to initially `rake congress:update` (perhaps as a part of db/seeds.rb?).

@lschatzkin can you take a look at this?

I can trigger a white screen after the first swipe by changing my language settings to something non-US.

Yeah I think it would be best to link directly to the same feed SEC is using itself: https://www.eff.org/deeplinks.xml?field_issue_tid=11461

Partially completed in #581 and #583. Currently the link to the markdown format can only be seen by admin users -- we should make them public. It would also be...

Aah we need to wait for sprockets 4.0 https://github.com/rails/sprockets#upgrading-to-sprockets-4x