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

Replace ppn hyperlink on the apply_confirm page #17

Open
cronchie opened this issue Feb 20, 2015 · 9 comments
Open

Replace ppn hyperlink on the apply_confirm page #17

cronchie opened this issue Feb 20, 2015 · 9 comments

Comments

@cronchie
Copy link
Contributor

image

Replace the hyperlink to Cook County site with local equivalent.

@cwang912
Copy link
Contributor

cwang912 commented Mar 3, 2015

I'm commenting the lines out -- I don't think it matters for our version, since we only care about the delinquent taxes for the owners property and not the lot they are applying for.

@cwang912
Copy link
Contributor

cwang912 commented Apr 1, 2015

Is this working? I feel like it is...

@pkoepke
Copy link
Contributor

pkoepke commented Jun 24, 2015

Just tested - still not working. The HTML is commented out so the user dues not see the link, but it's still sent to the end user. And it's currently pointing to the Cook Count website so we need to point it to the Cuyahoga County website.

This link is working on the Apply page (see issue 27) so we just need to plug in similar code.

@pkoepke
Copy link
Contributor

pkoepke commented Jun 24, 2015

Example of a working link to the Cuyahoga County website: http://clevelandlots.org/apply/#/browse?parcel=106-01-016

@girricane
Copy link

Additionally, the broken hyperlink is due to a similar issue of linking back to a Cook County site.

image

@girricane
Copy link

The GIS of Cuyahoga might have actual photos of the parcels, http://gis.cuyahogacounty.us/en-US/GIS-Data.aspx, but getting it might be more difficult as I couldn't find their images publicly available (seem to be only by request).

@eamonjohnson
Copy link
Contributor

Fixes in 45cb762

@eamonjohnson
Copy link
Contributor

@arielkoiman please review

@cwang912
Copy link
Contributor

Any updates on this?

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

5 participants