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 Netlify deployment #2

Closed
wants to merge 1 commit into from

Conversation

nwtgck
Copy link

@nwtgck nwtgck commented Feb 22, 2020

No description provided.

@nwtgck
Copy link
Author

nwtgck commented Feb 22, 2020

And we have another problem "Missing access token" below in the actions.
image
(see: https://github.com/kaihendry/dabase.com/pull/2/checks?check_run_id=461553156)

@kaihendry
Copy link
Owner

"GitHub automatically creates a GITHUB_TOKEN secret to use in your workflow." .... no? https://help.github.com/en/actions/configuring-and-managing-workflows/authenticating-with-the-github_token

@nwtgck
Copy link
Author

nwtgck commented Feb 22, 2020

@kaihendry Yes. But "Missing access token" is about Netlify token I think.

@kaihendry
Copy link
Owner

It's all set I think:

image

@kaihendry
Copy link
Owner

Appears to work on a test I made here: #3 (comment)

Perhaps some permission problem? Idk

@nwtgck
Copy link
Author

nwtgck commented Feb 22, 2020

@kaihendry It seems well...
I don't figure out the reason. Could you make sure NETLIFY_AUTH_TOKEN and NETLIFY_SITE_ID are the right values?

My repositories and other author repositories using actions-netlify work well.

@nwtgck
Copy link
Author

nwtgck commented Feb 22, 2020

Appears to work on a test I made here: #3 (comment)

Perhaps some permission problem? Idk

Great! It seems some permission problem.

@kaihendry
Copy link
Owner

Thank you for your help.

Perhaps it's because you modified the workflow and Github actions doesn't like that?

Thanks again!

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.

None yet

2 participants