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

fix: the vercel project name was showing as undefined #313

Merged
merged 2 commits into from
Nov 27, 2024

Conversation

mozisan
Copy link
Contributor

@mozisan mozisan commented Nov 20, 2024

初めまして。
このGitHub Actionsを便利に利用させていただいています🙏

利用させていただいている中で、1点だけ問題に遭遇しました。
私の利用している環境では、PRに自動で投稿されるコメントにおいて、Vercelのプロジェクト名が undefined になってしまっていました😞
fetchProjectName で利用しているVercelのAPIについて調べてみると、クエリパラメータとして teamId を受け付けることが分かりました。

ドキュメントの該当箇所

ドキュメントのスクリーンショット

CleanShot 2024-11-20 at 19 43 59

試しに同じAPIを手元のRESTクライアントで叩いてみたところ、以下の結果になることが分かりました。

前提条件:teamId をパラメータとして含めるかどうか以外は、このActionと同じ条件にする

チーム種別 teamIdなし teamIdあり
個人アカウントに紐づいて自動的に作成されるチーム
Create Teamを実行して作成したチーム

上記の結果を踏まえると、私が遭遇した「チーム名が undefined になる」という現象は、私がデプロイしようとしたプロジェクトが個人用のチームではなかったからだと考えられます。
また、個人用のチームでも teamId をパラメータに含めても特にエラーになるわけではなかったことから、このPRでは teamId パラメータを常に含むようにしました。

お手隙でご確認いただけると幸いです🙏

@InkoHX
Copy link
Member

InkoHX commented Nov 27, 2024

@mozisan 分かりやすいご説明と、PRありがとうございます👀

CIは失敗してますが、GitHub側のセキュリティ対策上ForkからのPRではGITHUB_TOKENの権限が削がれてしまい、実行できなくなっているようなので気にしなくて大丈夫です!

確認が遅くなっていまい、申し訳ありません🙏
ありがたくマージさせていただきます!

@InkoHX InkoHX merged commit 6617a98 into nexterias:main Nov 27, 2024
6 of 9 checks passed
@InkoHX
Copy link
Member

InkoHX commented Nov 27, 2024

@mozisan さっそくですが、v1.2.3としてリリースしました!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants