Skip to content

Conversation

@chouchouji
Copy link
Contributor

@chouchouji chouchouji commented Aug 22, 2025

Hi @rijkvanzanten, I am using this component. It is very great. I find there is a typo in readme and am pleased to fix it. Please review this pr when you have time. Thanks!

@chouchouji
Copy link
Contributor Author

chouchouji commented Aug 22, 2025

The ci failed. Can you tell me how to fix it? @rijkvanzanten Thanks!

@rijkvanzanten
Copy link
Member

Not sure! Lets take a look 🙂

Error is saying Error: Resource not accessible by integration

@rijkvanzanten
Copy link
Member

Ahh maybe the workflow needs permissions to accept writing the label from a fork PR. Lemme try something..

@rijkvanzanten
Copy link
Member

OK that did the trick! All systems green now 🙂

@chouchouji
Copy link
Contributor Author

It seems normal. Thanks for your work.

@rijkvanzanten rijkvanzanten merged commit 6abc780 into directus:main Aug 22, 2025
2 checks passed
@rijkvanzanten
Copy link
Member

Thank you for your PR! 🫶

@chouchouji chouchouji deleted the docs-readme branch August 22, 2025 13:12
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.

2 participants