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

Link to checklist instead of duplicating list #247

Closed
wants to merge 1 commit into from

Conversation

brittag
Copy link
Member

@brittag brittag commented Aug 7, 2016

Closes #138 - our member privileges onboarding/offboarding checklist needs to change regularly (as we add or remove services we use as DU), so we should link to that Google Doc instead of trying to make this bit of the app super helpful.

@lilliealbert
Copy link
Member

For links to google docs, I think I'd rather they be configurable than be hardcoded. That means you'd set them from Admin Tools --> Configure App page. I'll go ahead and do that this time and put in a link to it so you can use it as a pattern if you want to add any more links in the future.

@lilliealbert
Copy link
Member

Here's the commit that I implemented this in: ee5cca1

To make something configurable you:

  • Add it to the configurable.yml file:
cool_link:
  type: text
  • Reference it somewhere:
#{ link_to "a cool link", Configurable[:cool_link] }

Then, after the code is deployed, set the value in the UI (Admin Tools --> Configure App page)!

@brittag brittag deleted the link-checklist branch August 7, 2016 19:59
@brittag
Copy link
Member Author

brittag commented Aug 7, 2016

Neato, that makes sense. Thanks! I put this on https://github.com/doubleunion/arooo/wiki/How-to-add-a-configurable-link for reference in the future...if I remember it's there. :D

@lilliealbert
Copy link
Member

brilliant! 🎉

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

Successfully merging this pull request may close these issues.

2 participants