docs: removing external links for unified nav#2625
Conversation
Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2625 +/- ##
=======================================
Coverage 88.88% 88.88%
=======================================
Files 62 62
Lines 10238 10238
Branches 451 451
=======================================
Hits 9100 9100
Misses 1117 1117
Partials 21 21
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
zimeg
left a comment
There was a problem hiding this comment.
@haleychaas LGTM! Same ordering suggestion as the Bolt for Python PR - is moving "Release notes" before "Contributing" alright for a first read?
- Getting help
- Release notes
- Contributing
Feel free to merge whenever best! 📰
Summary
I'm working on uniting the nav bar for all of the tools so that you don't lose the context of the rest of the nav when you click on one of the tools. In this PR, I'm removing the external links (release notes, github repo, contributor's guide) to cut down on bloat in the nav menu when they are all united. Since the github repo and contributor's guide are already mentioned in the landing page, I just added the release notes link to that.
Preview for one nav here: https://docs-slack-d-unified-na-idfrpr.herokuapp.com/tools
PR for unified nav here: https://github.com/slackapi/docs/pull/605
Requirements