Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fuzzyEmail を無効にする #513

Merged
merged 1 commit into from
Oct 28, 2024
Merged

Conversation

cm-wada-yusuke
Copy link
Member

📑 Summary

markdown変換で、メールアドレスを自動で検知して mailto: のリンクに変換しているが、Zennオリジナルのリンクカード化が意図せず動いてしまいうまく機能していない。具体的には以下のように変換されてしまう。

# marldown
[email protected]

# html
URLが不正です

そもそもメールアドレスを検知して <a mailto="[email protected]">[email protected]</a> のように変換する処理が不要と考えたため、本機能を無効にします。

📋 Tasks

プルリクエストを作成いただく際、お手数ですが以下の内容についてご確認をお願いします。

  • 📖 Contribution Guide を読んだ
  • 👩‍💻 canary ブランチに対するプルリクエストである
  • zenn-cli で実行して正しく動作しているか確認する
  • 不要なコードが含まれていないか( コメントやログの消し忘れに注意 )
  • XSS になるようなコードが含まれていないか
  • モバイル端末での表示が考慮されているか
  • Pull Request の内容は妥当か( 膨らみすぎてないか )

より詳しい内容は Pull Request Policy を参照してください。

@cm-wada-yusuke cm-wada-yusuke added the feature 機能追加・改善 label Oct 22, 2024
Copy link
Contributor

@cm-dyoshikawa cm-dyoshikawa left a comment

Choose a reason for hiding this comment

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

LGTM!

@cm-wada-yusuke cm-wada-yusuke merged commit aa20fe2 into canary Oct 28, 2024
3 of 4 checks passed
@cm-wada-yusuke cm-wada-yusuke deleted the feat-disable-fuzzyEmail branch October 28, 2024 01:13
@cm-wada-yusuke cm-wada-yusuke mentioned this pull request Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 機能追加・改善
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants