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

Codesandbox URL に + を許容する #491

Merged
merged 1 commit into from Feb 5, 2024

Conversation

cm-igarashi-ryosuke
Copy link
Member

📑 Summary

CodeSandboxのembed URL に + を許可するように修正。

issue: #490

📋 Tasks

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

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

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

@cm-igarashi-ryosuke cm-igarashi-ryosuke added the feature 機能追加・改善 label Feb 5, 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-igarashi-ryosuke cm-igarashi-ryosuke merged commit 1e774a7 into canary Feb 5, 2024
4 checks passed
@cm-igarashi-ryosuke cm-igarashi-ryosuke deleted the fix-codesandbox-url-validation branch February 5, 2024 08:27
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.

None yet

2 participants