Skip to content

Link OWASP Community text to Slack in BLT University footer#36

Merged
DonnieBLT merged 2 commits intoOWASP-BLT:mainfrom
Manahil-Afzal:footer-owasp-link
Mar 26, 2026
Merged

Link OWASP Community text to Slack in BLT University footer#36
DonnieBLT merged 2 commits intoOWASP-BLT:mainfrom
Manahil-Afzal:footer-owasp-link

Conversation

@Manahil-Afzal
Copy link
Copy Markdown
Contributor

@Manahil-Afzal Manahil-Afzal commented Mar 25, 2026

What this PR does:

Updates the BLT University site footer to make the “OWASP Community” text clickable.
Links it directly to the OWASP Slack invite
channel.

Why this is needed:

Makes it easier for visitors to join the OWASP community.
Improves site interactivity and accessibility.

Files changed:

_includes/footer.html

No other changes were made; only the footer link was updated.
This PR is ready for review and merge.

Summary by CodeRabbit

  • Improvements
    • Updated footer "OWASP Community" link to use expanded markup with descriptive aria-label and title for better accessibility.
    • Added hover underline styling for clearer visual feedback on the link.
    • Adjusted link text placement and surrounding spacing for cleaner markup and ensured the template ends with a newline.

@owasp-blt
Copy link
Copy Markdown

owasp-blt bot commented Mar 25, 2026

👋 Hi @Manahil-Afzal!

This pull request needs a peer review before it can be merged. Please request a review from a team member who is not:

  • The PR author
  • coderabbitai
  • copilot

Once a valid peer review is submitted, this check will pass automatically. Thank you!

⚠️ Peer review enforcement is active.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 25, 2026

Warning

Rate limit exceeded

@Manahil-Afzal has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 18 minutes and 57 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository: OWASP-BLT/coderabbit/.coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: e7b815cf-61fa-4d9c-b822-7175e32c97c2

📥 Commits

Reviewing files that changed from the base of the PR and between 93c41bb and cfee667.

📒 Files selected for processing (1)
  • _includes/footer.html

Walkthrough

Replaced the footer’s single-line anchor with a multi-line <a> in _includes/footer.html, adding aria-label, title, hover:underline, adjusted visible text spacing, and ensured the file ends with a newline.

Changes

Cohort / File(s) Summary
Footer Template
\_includes/footer.html
Rewrote the footer anchor markup: replaced the single-line <a> with a multi-line <a> that adds aria-label and title attributes, introduces hover:underline, tweaks whitespace around "OWASP Community", and ensures a trailing newline at EOF.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels: quality: medium

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title states the PR links OWASP Community text to Slack, but the actual change only adds hover styling and accessibility attributes without confirming a Slack link is present in the footer change. Verify whether the href attribute in the footer update actually points to the Slack invite URL. If it does, the title is accurate; if not, update the title to reflect the actual changes made (e.g., 'Improve OWASP Community link accessibility in footer').
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@S3DFX-CYBER S3DFX-CYBER left a comment

Choose a reason for hiding this comment

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

LGTM

@Manahil-Afzal
Copy link
Copy Markdown
Contributor Author

@DonnieBLT review this please

@DonnieBLT DonnieBLT merged commit a1e52b1 into OWASP-BLT:main Mar 26, 2026
1 check passed
@owasp-blt
Copy link
Copy Markdown

owasp-blt bot commented Mar 26, 2026

🎉 PR merged! Thanks for your contribution, @Manahil-Afzal!

Your work is now part of the project. Keep contributing to OWASP BLT-Pool and help make the web a safer place! 🛡️

Visit pool.owaspblt.org to explore the mentor pool and connect with contributors.


📊 Monthly Leaderboard

Hi @Manahil-Afzal! Here's how you rank for March 2026:

Rank User Open PRs PRs (merged) PRs (closed) Reviews Comments Total
52 haroon0x @haroon0x 1 1 0 0 2 15
53 Manahil-Afzal @Manahil-Afzal 1 1 2 0 4 15
54 renarin-kholin @renarin-kholin 0 0 0 3 0 15

Scoring this month (across OWASP-BLT org): Open PRs (+1 each), Merged PRs (+10), Closed (not merged) (−2), Reviews (+5; first two per PR in-month), Comments (+2, excludes CodeRabbit). Run /leaderboard on any issue or PR to see your rank!


🔍 Reviewer Leaderboard

Top reviewers for March 2026 (across the OWASP-BLT org):

Rank Reviewer Reviews this month
🥈 2 Nachiket-Roy @Nachiket-Roy 40
🥉 3 Jayant2908 @Jayant2908 39
4 S3DFX-CYBER @S3DFX-CYBER 33
5 arnavkirti @arnavkirti 26
6 e-esakman @e-esakman 17

Reviews earn +5 points each in the monthly leaderboard (first two reviewers per PR). Thank you to everyone who helps review PRs! 🙏

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants