Skip to content

Document URL encoding requirement for mailto query parameters#35

Merged
umputun merged 1 commit intomasterfrom
docs/mailto-url-encoding-note
Feb 11, 2026
Merged

Document URL encoding requirement for mailto query parameters#35
umputun merged 1 commit intomasterfrom
docs/mailto-url-encoding-note

Conversation

@paskal
Copy link
Copy Markdown
Collaborator

@paskal paskal commented Feb 11, 2026

Summary

  • Added note to Send godoc about URL-encoding query parameter values in mailto URLs
  • Added note to README email section
  • Specifically calls out the + character being interpreted as a space if not encoded as %2B

Relates to umputun/remark42#1946

@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 21887439528

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.4%) to 96.721%

Totals Coverage Status
Change from base Build 19937596487: 0.4%
Covered Lines: 472
Relevant Lines: 488

💛 - Coveralls

@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 21887438389

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.4%) to 96.721%

Totals Coverage Status
Change from base Build 19937596487: 0.4%
Covered Lines: 472
Relevant Lines: 488

💛 - Coveralls

@paskal paskal marked this pull request as ready for review February 11, 2026 00:14
@paskal paskal requested a review from umputun as a code owner February 11, 2026 00:14
Copy link
Copy Markdown
Member

@umputun umputun left a comment

Choose a reason for hiding this comment

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

lgtm, thx

@umputun umputun merged commit a02393f into master Feb 11, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants