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

Add Contribute page #148

Open
wants to merge 21 commits into
base: main
Choose a base branch
from

Conversation

vickytnz
Copy link
Contributor

@vickytnz vickytnz commented Nov 24, 2024

Added contribution page as a light touch version of the Service Manual pages.

This pull request includes:

  • the page
  • a link in the footer (for now)

contribution page

@vickytnz vickytnz linked an issue Nov 24, 2024 that may be closed by this pull request
@frankieroberto frankieroberto temporarily deployed to nhs-prototyp-92-create--8qo2uz November 24, 2024 17:51 Inactive
@vickytnz vickytnz temporarily deployed to nhs-prototyp-92-create--8qo2uz November 24, 2024 22:07 Inactive
@frankieroberto
Copy link
Contributor

@vickytnz I like this but wonder if it’s maybe a bit long? Can I have a go at cutting it down a bit, if you’re ready? (It's in Draft so wasn't sure if you want reviews yet or not).

Also not sure about the "Soon, you’ll be able to..." section - could we just add that stuff when it happens rather than making promises for the future?

@vickytnz
Copy link
Contributor Author

@frankieroberto yeah sounds good - will change this to a PR to change.

Agree that for the future stuff we can just say: 'we will add other things as the become available- if you have an idea contact us'

@frankieroberto
Copy link
Contributor

@vickytnz it's already a PR! 😂 I’ll hit the "Ready for review" button. 😄

@frankieroberto frankieroberto marked this pull request as ready for review November 25, 2024 21:37
@frankieroberto frankieroberto temporarily deployed to nhs-prototyp-92-create--8qo2uz November 25, 2024 21:38 Inactive
Copy link
Contributor

@frankieroberto frankieroberto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Made some initial suggestions!

app/views/contribute.html Outdated Show resolved Hide resolved
app/views/contribute.html Outdated Show resolved Hide resolved
app/views/contribute.html Outdated Show resolved Hide resolved
app/views/contribute.html Outdated Show resolved Hide resolved
app/views/contribute.html Outdated Show resolved Hide resolved
app/views/contribute.html Outdated Show resolved Hide resolved
app/views/contribute.html Outdated Show resolved Hide resolved
app/views/contribute.html Outdated Show resolved Hide resolved
app/views/contribute.html Outdated Show resolved Hide resolved
app/views/contribute.html Outdated Show resolved Hide resolved
vickytnz and others added 3 commits November 26, 2024 15:32
Co-authored-by: Frankie Roberto <frankie@frankieroberto.com>
Co-authored-by: Frankie Roberto <frankie@frankieroberto.com>
Co-authored-by: Frankie Roberto <frankie@frankieroberto.com>
@vickytnz vickytnz temporarily deployed to nhs-prototyp-92-create--8qo2uz November 26, 2024 15:33 Inactive
I will remove but am half waiting for this to be a thing as I have experienced in a lot of places that people on Surface Pros have Github blocked, let's see if this becomes a thing

Co-authored-by: Frankie Roberto <frankie@frankieroberto.com>
@vickytnz vickytnz temporarily deployed to nhs-prototyp-92-create--8qo2uz November 26, 2024 15:36 Inactive
Co-authored-by: Frankie Roberto <frankie@frankieroberto.com>
@vickytnz vickytnz temporarily deployed to nhs-prototyp-92-create--8qo2uz November 26, 2024 15:37 Inactive
Co-authored-by: Frankie Roberto <frankie@frankieroberto.com>
@vickytnz vickytnz temporarily deployed to nhs-prototyp-92-create--8qo2uz November 26, 2024 15:37 Inactive
vickytnz and others added 2 commits November 26, 2024 19:03
Co-authored-by: Frankie Roberto <frankie@frankieroberto.com>
Co-authored-by: Frankie Roberto <frankie@frankieroberto.com>
@vickytnz vickytnz temporarily deployed to nhs-prototyp-92-create--8qo2uz November 26, 2024 19:03 Inactive
Co-authored-by: Frankie Roberto <frankie@frankieroberto.com>
@vickytnz vickytnz temporarily deployed to nhs-prototyp-92-create--8qo2uz November 26, 2024 19:04 Inactive
@vickytnz vickytnz temporarily deployed to nhs-prototyp-92-create--8qo2uz November 26, 2024 20:00 Inactive
@vickytnz
Copy link
Contributor Author

Changes

contribution page with sections why contribute to the kit, how you can contribute to the kit and get started with a contribution

Copy link
Contributor

@frankieroberto frankieroberto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is looking good! A few more suggestions (feel free to ignore if you disagree with any!)

app/views/contribute.html Outdated Show resolved Hide resolved
app/views/contribute.html Outdated Show resolved Hide resolved
app/views/contribute.html Outdated Show resolved Hide resolved
app/views/contribute.html Outdated Show resolved Hide resolved
app/views/contribute.html Outdated Show resolved Hide resolved
app/views/contribute.html Outdated Show resolved Hide resolved
app/views/contribute.html Outdated Show resolved Hide resolved
app/views/contribute.html Outdated Show resolved Hide resolved
app/views/contribute.html Outdated Show resolved Hide resolved
app/views/contribute.html Outdated Show resolved Hide resolved
@frankieroberto frankieroberto changed the title First pass of contribute page and link Add Contribute page Nov 26, 2024
Co-authored-by: Frankie Roberto <frankie@frankieroberto.com>
@vickytnz vickytnz temporarily deployed to nhs-prototyp-92-create--8qo2uz November 26, 2024 22:22 Inactive
@frankieroberto frankieroberto mentioned this pull request Nov 26, 2024
vickytnz and others added 2 commits November 26, 2024 23:16
Co-authored-by: Frankie Roberto <frankie@frankieroberto.com>
Co-authored-by: Frankie Roberto <frankie@frankieroberto.com>
@vickytnz vickytnz temporarily deployed to nhs-prototyp-92-create--8qo2uz November 26, 2024 23:17 Inactive
Co-authored-by: Frankie Roberto <frankie@frankieroberto.com>
@vickytnz vickytnz temporarily deployed to nhs-prototyp-92-create--8qo2uz November 26, 2024 23:33 Inactive
@vickytnz vickytnz temporarily deployed to nhs-prototyp-92-create--8qo2uz November 26, 2024 23:48 Inactive
@vickytnz
Copy link
Contributor Author

vickytnz commented Nov 26, 2024

Final tweaks

I have also removed all of the 'add your own' text from each section.

contribute model with sections

@vickytnz vickytnz temporarily deployed to nhs-prototyp-92-create--8qo2uz November 26, 2024 23:53 Inactive
frankieroberto added a commit that referenced this pull request Nov 27, 2024
This adds a "primary nav" with 3 items:

* Get started
* Guides
* Support

"Contribute" could be added when #148 is merged.

I’ve not included "Page templates" as that page feels less valuable
longer-term (eg if the NHS website templates get moved to a plugin or
something).

Adding the primary nav possibly means that some of the breadcrumbs could
be removed or shortened (eg the "Home" breadcrumb link on Support feels
a bit odd now?), but the Service Manual website has both, so maybe it’s
fine?

There’s no "current section" indicator for now, but we can adopt that
when it gets added to the header in the next major release of NHS
frontend (hopefully).

Part of #93
@vickytnz vickytnz added the Content Improvements or additions to documentation label Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create a standalone page for Contributing
2 participants