Closed rooneywp closed 8 years ago
@rooneywp This one isn't yet ready for review, correct?
@rooneywp Given our conversation in Slack, I'll assume you'd like me to review this.
Two initial things:
18f-pages
branch, so things are a bit muddled at the moment. Let's sort those out, merge them into this branch, push it to staging, and then we'll be able to confirm we're good to go.@fureigh Yeah, I thought something was weird. I must have done something wrong when I was creating the new files.
I'll update the benefits links.
@fureigh Don't worry about rushing to get this done tonight. We can get it up in the AM.
@rooneywp Patience appreciated, but I'm in workshop mode for the rest of this week, so best to wrap it up tonight if possible. Aiming to get the branches sorted before this plane lands, after which you can make further edits at your convenience.
Merged branch rooneywp-patch-6
.
Okay, branches appear to be sorted. I pushed to staging but 2FA is locking me out while I'm on a plane and don't have the ability to access texts. Problems of living in the future!
@rooneywp, can you review on staging? In particular, please verify that the links all work and are formatted correctly.
No pressure to do that tonight, of course; it's up to you. Everything is now in this PR and on staging, including the commits that were already merged to 18f-pages
.
IN PROGRESS
Adding the messaging for the new section to house the RaaS roles & links to position descriptions