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

Update nav components for 9.0 announce #6646

Merged
merged 6 commits into from
Sep 25, 2023
Merged

Update nav components for 9.0 announce #6646

merged 6 commits into from
Sep 25, 2023

Conversation

emdecr
Copy link
Contributor

@emdecr emdecr commented Sep 15, 2023

Summary

This PR adds the 9.0 announce blog to the notification banner (set for 2 weeks), and updates the global nav to align with mm.com.

@cwarnermm – Feel free to adjust the footer links and I'll mirror the changes on mm.com.

Also, I am going to put a DNM tag on until I can chat with Aaron about the fallback link once two weeks is up. Should be taken off early this morning.

Thank you!

@emdecr emdecr added Do Not Merge Should not be merged until this label is removed 2: Editor Review Requires review by an editor labels Sep 15, 2023
@emdecr emdecr requested a review from cwarnermm September 15, 2023 12:23
@emdecr emdecr self-assigned this Sep 15, 2023
@github-actions
Copy link

Newest code from emdecr has been published to preview environment for Git SHA 23f8f12

1 similar comment
@github-actions
Copy link

Newest code from emdecr has been published to preview environment for Git SHA 23f8f12

@github-actions
Copy link

Newest code from emdecr has been published to preview environment for Git SHA 2007bcc

@emdecr emdecr removed the Do Not Merge Should not be merged until this label is removed label Sep 15, 2023
@emdecr
Copy link
Contributor Author

emdecr commented Sep 15, 2023

AT approved the copy and timeframe, so we're all good with all that~

@cwarnermm
Copy link
Member

There's a few links we may want to revisit and a few we may want to realign with our messaging house. There's a few cases, noted below, where I'm not sure whether we want to adjust the website or leave as-is. All input is welcome.

In the header:

  • Let's update the Academy link to point to: https://academy.mattermost.com/
  • [INPUT NEEDED] For the Release Notes link, we can keep it pointing to the self-hosted changelog OR we could update it to point to the changelog landing page instead: https://docs.mattermost.com/guides/changelogs.html. I'm open to either.

In the footer:

  • The All Docs link looks like it's missing the https://docs.mattermost.com/ link, but that might be something auto-generated.
  • [INPUT NEEDED] The Admin link redirects as expected. Do we want to keep it as Admin or update it to Manage or similar?
  • [INPUT NEEDED] The Playbooks link redirects as expected. In the docs, we're de-emphasizing the vertical name Mattermost Playbooks in favour of the outcome building repeatable processes. Not sure if we want to make a similar update to our terminology here.
  • [INPUT NEEDED] The Channels link redirects as expected. In the docs, we're de-emphasizing the vertical name in favour of the outcome connect and collaborate. This is another case where I'm not sure whether we want to do the same to the website.

Oh, and the Twitter logo is outdated.

@emdecr
Copy link
Contributor Author

emdecr commented Sep 15, 2023

Thanks @cwarnermm! Comments sub-bulleted.

In the header:

In the footer:

  • The All Docs link looks like it's missing the https://docs.mattermost.com/ link, but that might be something auto-generated.
    • Yep, I've got the paths set to the root, so it'll pick up on whatever the domain is.
  • [INPUT NEEDED] The Admin link redirects as expected. Do we want to keep it as Admin or update it to Manage or similar?
  • [INPUT NEEDED] The Playbooks link redirects as expected. In the docs, we're de-emphasizing the vertical name Mattermost Playbooks in favour of the outcome building repeatable processes. Not sure if we want to make a similar update to our terminology here.
  • [INPUT NEEDED] The Channels link redirects as expected. In the docs, we're de-emphasizing the vertical name in favour of the outcome connect and collaborate. This is another case where I'm not sure whether we want to do the same to the website.
    • Those three points are good flags. Can I ask the team and circle back in another PR with their changes, if any? We're still deciding on how we want to work in the de-emphasis of the vertical in favour of the outcomes, and a bunch of other messaging house changes (and how they affect structure and terminology).

Oh, and the Twitter logo is outdated.

  • I believe this one is still on hold because other orgs have not made the change yet either (ex. Atlassian, GitLab, etc.). Asked the team again and waiting on their feedback.

I can push up the checkmarks shortly.

@cwarnermm
Copy link
Member

@emdecr - Yes. A separate PR makes sense, given that there's discussion and alignment needed. Thanks!!

@github-actions
Copy link

Newest code from emdecr has been published to preview environment for Git SHA dc108ad

@github-actions
Copy link

Newest code from cwarnermm has been published to preview environment for Git SHA 820392d

@emdecr
Copy link
Contributor Author

emdecr commented Sep 18, 2023

@cwarnermm Just a heads up for this PR and the one for dev docs. I'm going to be OOO until next Monday starting tomorrow, but feel free to tag the marketing folks (AZ/Kevin/Aaron) if you need another pair of eyes for anything related.

Also update: We've decided to start updating the Twitter logo, but that will come in another PR once I sort a few things.

TY!

@github-actions
Copy link

Newest code from emdecr has been published to preview environment for Git SHA 4464c72

Copy link
Member

@cwarnermm cwarnermm left a comment

Choose a reason for hiding this comment

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

Thank you, @emdecr!!

@github-actions
Copy link

Newest code from cwarnermm has been published to preview environment for Git SHA 5ba40f7

@cwarnermm cwarnermm added 3: Reviews Complete All reviewers have approved the pull request and removed 2: Editor Review Requires review by an editor labels Sep 25, 2023
@cwarnermm cwarnermm merged commit 608fa8a into master Sep 25, 2023
3 checks passed
@cwarnermm cwarnermm deleted the header-update-9-0 branch September 25, 2023 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants