DomGarguilo commented on code in PR #415:
URL: https://github.com/apache/accumulo-website/pull/415#discussion_r1496237964
##########
_includes/nav.html:
##########
@@ -42,6 +42,7 @@
<li><a class="dropdown-item" href="{{ site.baseurl
}}/related-projects">Related Projects</a></li>
</ul>
</li>
+ <li class="nav-item"><a class="nav-link" href="{{ site.baseurl
}}/accumulo-access">Accumulo Access</a></li>
Review Comment:
Kinda seems out of place to add a new item to the nav bar just for accumulo
access. It could be cool to have a section to link to all of the adjacent
accumulo projects/repos with a brief description of what each is (like
accumulo-testing, access, docker, etc.). Not sure where the best place for that
is though and also might not be within the scope of this PR.
Just an idea too. Not recommending we add it, just wanted to get the idea
out there.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
