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.

Sign up for GitHub

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

Jump to bottom

Add language about our services #298

Merged
merged 3 commits into from
Sep 4, 2024
Merged

Add language about our services #298

merged 3 commits into from
Sep 4, 2024

Conversation

choldgraf
Copy link
Member

This does two main things:

  • Adds language + a link on our home page about us providing consulting services
  • Adds language to our platform page about what kinds of services we provide, with another CTA to the join page

@Gman0909 is this the kind of thing you had in mind?

@choldgraf choldgraf requested review from jnywong and Gman0909 August 22, 2024 21:35
Gman0909
Gman0909 requested changes Aug 23, 2024
View reviewed changes
Copy link
Contributor

@Gman0909 Gman0909 left a comment

Choose a reason for hiding this comment

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

I think this is looking good, besides the adjustments I suggested. I would also maybe run the new language by @jmunroe as he was the one who originally suggested we highlight our consultancy services on our brochure site.

content/_index.md Show resolved Hide resolved
content/platform/_index.md Outdated Show resolved Hide resolved
content/platform/_index.md Show resolved Hide resolved
jnywong
jnywong approved these changes Aug 29, 2024
View reviewed changes
Copy link
Member

@jnywong jnywong left a comment

Choose a reason for hiding this comment

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

lgtm

@aprilmj aprilmj assigned jmunroe and unassigned jmunroe Sep 3, 2024
@jmunroe jmunroe self-assigned this Sep 3, 2024
@aprilmj aprilmj requested a review from jmunroe September 3, 2024 16:30
@jmunroe jmunroe removed their assignment Sep 3, 2024
jmunroe
jmunroe approved these changes Sep 4, 2024
View reviewed changes
Copy link
Contributor

@jmunroe jmunroe left a comment

Choose a reason for hiding this comment

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

All good to go.

@choldgraf choldgraf merged commit 883d1a2 into main Sep 4, 2024
2 checks passed
@choldgraf choldgraf deleted the services branch September 4, 2024 19:44
@choldgraf
Copy link
Member Author

thanks for the feedback all - let's keep iterating on this content as we learn more

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gman0909 Gman0909 Gman0909 requested changes

@jmunroe jmunroe jmunroe approved these changes

@jnywong jnywong jnywong approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@choldgraf @jmunroe @Gman0909 @jnywong