-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add more metadata #35
Comments
Hi there! @lizzthabet and I are interested in helping with this issue if you're still looking for contributors! We have a couple questions for the best way to proceed:
This seemed like a good first issue to get started with, but if there are other issues that are more pressing just let us know! Thanks 😄 |
hey @jaqarrick thanks for taking a look at this! Sorry for my slow response - been a little busy with the holiday to answer your questions:
thanks for your interest in this project! let me know if there's anything else I can do to help
|
Hi Ross, Thanks for getting back to us! We got started on this issue today and added the text field to display next to the phone number. @lizzthabet started a PR. As far as interactivity, we thought it would be nice for the text and website options to each display in their own svg button/icon next to the telephone icon. Are you using a SVG/Icon library and if so, are there icons already available to represent 'text' and 'website'? Thanks! Jack |
…so that texting can be interactive in the future
…each method of contacting hotlines and link icons to SMS and website URLS
Hey @jaqarrick sorry for my slow response => all the icons were done by a volunteer designer. But anything that is in the same vein should be fine! |
…each method of contacting hotlines and link icons to SMS and website URLS
…each method of contacting hotlines and link icons to SMS and website URLS
@rossPatton, thanks! We ended using open source icons from Bootstrap. The PR #51 is ready for review, if you want to take a look and let us know what next steps are. Any feedback is appreciated! |
* Issue #35: Add SMS text field to display for each hotline * Issue #35: Refactor data structure of text contact info so that texting can be interactive in the future * Issue #35: Add IconGroup component to display icons for each method of contacting hotlines and link icons to SMS and website URLS * No linked issue: replace en dash with hyphen in phone number hotline constants Co-authored-by: jaqarrick <[email protected]>
Currently we attach some metadata to each hotline entry, but it's not consistent and most of it is not used. Currently the only data we render on hotline pages are the hotline name, phone number and hours of operation and whether it's a crisis hotline or not (by color)
We should make sure all hotline entries also include the following metadata:
The text was updated successfully, but these errors were encountered: