Skip to content

feat(email): show trending Skill descriptions - #420

Merged
harlan-zw merged 1 commit into
mainfrom
fix/trending-email-descriptions
Oct 5, 2026
Merged

harlan-zw merged 1 commit into
mainfrom
fix/trending-email-descriptions

Conversation

@harlan-zw

Copy link
Copy Markdown
Collaborator

🔗 Linked issue

❓ Type of change

  • 📖 Documentation
  • 🐞 Bug fix
  • 👌 Enhancement
  • ✨ New feature
  • 🧹 Chore
  • ⚠️ Breaking change

📚 Description

The trending Skills email hides descriptions in HTML. Show each description below its Skill name, using the existing length limit and HTML escaping.

🤖 AI disclosure: Harlan Agent Kit modified this description. My AI open-source policy.

@harlan-zw
harlan-zw merged commit 0bce5a5 into main Oct 5, 2026
6 checks passed
@harlan-zw harlan-zw added the harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. label Oct 5, 2026
@harlan-zw

Copy link
Copy Markdown
Collaborator Author

Agent verification: inspected the rendered email on desktop and mobile, in light and dark themes. Descriptions wrap below Skill names. Missing descriptions leave no blank line.

trending-email-desktop

trending-email-mobile

trending-email-dark

@harlan-github-agent harlan-github-agent Bot added harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. and removed harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. harlan-agent-running An Agent holds a Task on this issue or pull request right now. labels Oct 5, 2026
@harlan-zw
harlan-zw deleted the fix/trending-email-descriptions branch October 5, 2026 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant