Repository navigation
Update public credential copy - #3
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
There was a problem hiding this comment.
🟡 Changes recommended
The structured data and redirect implementation include correctness/SEO and privacy concerns (non-schema JSON-LD properties, host-relative redirect target, and publishing an unnecessary external ID).
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This PR updates the site’s public-facing credential/SEO copy to present CompTIA SecurityX as the lead current certification, removes CISSP/SSCP from “current” mentions, and adds a static /resume redirect to the About page to consolidate outdated URLs.
Changes:
- Update homepage/About/Services/Contact copy to reflect current vs expired credentials and remove clearance/public-trust advertising language.
- Expand structured-data (JSON-LD) credentials to CompTIA-only and update meta descriptions accordingly.
- Add
/resume→/about.htmlredirect page and refresh “canonical host / outdated URLs” messaging (README + 404).
File summaries
| File | Description |
|---|---|
| stylesheet.css | Adds spacing rule after credential list for improved layout. |
| services.html | Updates toolkit/constraints copy (expired certs + removes “facility clearance”). |
| resume/index.html | Adds client-side redirect page for /resume to About. |
| README.md | Updates canonical/outdated host and URL guidance. |
| index.html | Updates meta description, JSON-LD credentials, and homepage hero credential line. |
| contact.html | Adjusts placeholder text and updates footer note about sponsor-paid cert sittings. |
| about.html | Reworks “Credentials” section to CompTIA-only current list and moves CISSP/SSCP to expired mention. |
| 404.html | Adds About link and note about old /resume links. |
Review details
Suppressed comments (1)
resume/index.html:11
- The JavaScript redirect also points to
/about.htmlon the current host, which can keep users on an outdated host and overrides the meta refresh. Consider redirecting directly to the canonical URL.
<script>location.replace("/about.html");</script>
- Files reviewed: 8/8 changed files
- Comments generated: 3
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| "recognizedBy": { "@type": "Organization", "name": "CompTIA" }, | ||
| "validFrom": "2024-05-20", | ||
| "validThrough": "2027-05-20" |
| <head> | ||
| <meta charset="UTF-8"> | ||
| <meta name="viewport" content="width=device-width, initial-scale=1.0"> | ||
| <meta http-equiv="refresh" content="0; url=/about.html"> |
| <p>B.S., Information Systems Management — University of Maryland Global Campus. A.S., Electronic Systems Technology — Community College of the Air Force.</p> | ||
| <p class="small">SANS cybersecurity coursework. Short courses (MIT, FIRST CVSS v4.0, Digital Forensics Essentials) stay in the training column, not the cert column.</p> | ||
| <p class="small">I previously held Microsoft, Cisco, Juniper, and AWS certifications. They are expired, so they are not listed as current. The underlying work — enterprise routing, GovCloud, and operations — is still in scope.</p> | ||
| <p class="small">I previously held CISSP (expired August 2024), SSCP (expired June 2025), Microsoft, Cisco, Juniper, and AWS certifications. They are expired, so they are not listed as current. (ISC)² Non-Member ID 444470. The underlying work — enterprise routing, GovCloud, and operations — is still in scope.</p> |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 32c1e14. Configure here.
| </li> | ||
| <li> | ||
| <strong>CompTIA Secure Infrastructure Specialist (CSIS)</strong> | ||
| <span class="small">Issued September 2009 · does not expire</span> |
There was a problem hiding this comment.
Ineligible stackable certs marked current
High Severity
The Current list presents CIOS and CSIS as 2009 lifetime certs. CompTIA stackable designations launched in 2017 and require active CE credentials, so they do not follow from the lifetime A+, Network+, and Security+ already listed. The homepage hero and JSON-LD hasCredential block repeat that claim.
Additional Locations (2)
Reviewed by Cursor Bugbot for commit 32c1e14. Configure here.
Summary
/resumeat About; treat other hosts as outdatedTest plan
/resumelands on AboutNote
Low Risk
Static HTML/CSS copy and a client-side redirect only; no auth, APIs, or data handling. Main review surface is factual accuracy of certification claims and SEO/structured-data consistency.
Overview
Updates marketing and credential copy so CompTIA SecurityX is the lead current certification and lifetime CompTIA certs appear under Current on About; CISSP and SSCP are removed from hero text, meta descriptions, and JSON-LD and are only mentioned as expired in education/history copy.
Adds
/resume→/about.htmlredirect (resume/index.htmlwith meta refresh, canonical, andnoindex) and expands canonical-domain messaging in README, About banner, and 404 (About link plus note that old/resumebelongs on About).Copy tweaks elsewhere: drops Public Trust and clearance / facility clearance references from home hero and contact/services placeholders; notes openness to sponsor-paid cert exams on About and Contact; small CSS spacing after the credentials list.
Reviewed by Cursor Bugbot for commit 32c1e14. Bugbot is set up for automated code reviews on this repo. Configure here.