-
Notifications
You must be signed in to change notification settings - Fork 1
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
Added support page and link and tweaked the about page #149
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Made some suggestions.
Co-authored-by: Frankie Roberto <[email protected]>
Co-authored-by: Frankie Roberto <[email protected]>
Co-authored-by: Frankie Roberto <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All changes make sense
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All changes accepted
@vickytnz just realised the 2 slack links need different link text for accessibility reasons. Any ideas? |
@frankieroberto Ah yeah that's why in the original one I led with the Slack instance as is does make a difference for some people which one to use. I actually I think on the second we can get rid of the NHS service manual option.
We are supposed to follow the 'if ... then' format so if we need anything more than the header at the start then we probably need to roll back to explaining who can use which instance |
@vickytnz have tweaked slightly. How’s this: |
I have:
Final couple of tweaks that i've picked up after being away from it :
|
@vickytnz nice! Think we’re good to merge. 🚢 |
New support page
About page changed
Footer