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

User can click on, search for, route to Commercial Overlays. Details link to handbook info. #309

Closed
andycochran opened this issue Oct 11, 2017 · 7 comments

Comments

@andycochran
Copy link
Contributor

andycochran commented Oct 11, 2017

Need to get some summary text so we can display it on the route.

@andycochran
Copy link
Contributor Author

Use the tooltip for the summary text. Unblocking…

@andycochran andycochran added to do and removed Blocked An issue blocked by an external factor next sprint labels Oct 18, 2017
@andycochran andycochran assigned andycochran and unassigned fruchala Oct 19, 2017
@andycochran andycochran removed their assignment Oct 19, 2017
@andycochran
Copy link
Contributor Author

@allthesignals I think we should close this as wontfix. You can already get to Commercial Overlay details from the lot view. We need to establish rule for why something is routable. I don't think "so you can link externally" is a good reason for something to be routable. Special Purpose districts are routable because you can search for them, which is a good reason. Objections to closing this?

@fruchala
Copy link

fruchala commented Oct 20, 2017 via email

@allthesignals allthesignals reopened this Oct 20, 2017
@andycochran
Copy link
Contributor Author

andycochran commented Oct 20, 2017

Thanks, @fruchala. That's a good reason for making COs routable AND searchable.

@andycochran andycochran changed the title User can click on/route to Commercial Overlays. Details link to handbook info User can click on, search for, route to Commercial Overlays. Details link to handbook info. Oct 20, 2017
@allthesignals
Copy link
Collaborator

How should routing working for this? Is it feature-level or...?

@allthesignals
Copy link
Collaborator

Screen Shot 2017-11-20 at 1.48.50 PM.png

Any thoughts on the language for this view? I'll just link to commercial overlay?

@fruchala
Copy link

fruchala commented Nov 20, 2017 via email

allthesignals added a commit that referenced this issue Nov 20, 2017
Addresses #309 by making it routable.
This was referenced Nov 20, 2017
@ghost ghost removed the in progress label Nov 21, 2017
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

4 participants