-
Notifications
You must be signed in to change notification settings - Fork 149
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
GitHub Action Workflow fails due to denied permission #7
Comments
Hey Manuel, you are totally right. I added your suggestions to the workflow file and everything seems fine now. Regarding the issue with the file extensions: This is indeed they way it is supposed to work. The earlier versions, where this docker image also provided an I also have to admit that I'm not totally happy with this. The reason for the language based extension is that skohub-vocabs is supposed to run with an apache webserver. The apache will automatically fetch the right language based on your browser settings. So the suggested workaround for you would be to add If this somehow is not suitable, feel free to open a ticket in the skohub-vocabs repo, so I have a reason to work on it. Thanks! I will close this issue for now, feel free to reopen, if the issue remains. |
Thanks Steffen for clarifying the underlying issue. I think a major drawback could be, that the id of concepts cannot be used as an URI directly (which previously was the case), so it might be an idea for another ticket to deal with that issue (e.g. htaccess-wise with defaulting to first/one language?) Example:
I know that the concept ids main purpose is unique identification but from a users perspective some of them look like URIs where users might expect meaningful content. I will open an issue where the relevance of that could be discussed. |
Setup
docker image used in workflow:
skohub/skohub-vocabs-docker:latest
workflow file example: main.yml
github action: job error
❗ Error (job failed in stage)
❔ Possibile reason
❕ Possible solutions
✔️ Tested workaround
The text was updated successfully, but these errors were encountered: