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

Document how to make @authorize directives public #7458

Closed
wants to merge 1 commit into from

Conversation

Cyberboss
Copy link
Contributor

I was trying to get @authorize directives to show up in my schema. Found a workaround, putting this in docs to hopefully save others the 3 hour headache.

Should they be internal?

@CLAassistant
Copy link

CLAassistant commented Sep 13, 2024

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added the 📚 documentation This issue is about working on our documentation. label Sep 13, 2024
Cyberboss added a commit to tgstation/tgstation-server that referenced this pull request Sep 13, 2024
@Cyberboss
Copy link
Contributor Author

Cyberboss commented Sep 14, 2024

This may not be all that's necessary. I'm still not seeing it attached to fields though the definition is present now.

@Cyberboss Cyberboss marked this pull request as draft September 14, 2024 02:09
@Cyberboss
Copy link
Contributor Author

Seems like visualizing directives like this isn't a widespread feature even with EnableDirectiveIntrospection. Also the IntrospectionQuery in bananacakepop doesn't support it either.

@Cyberboss Cyberboss closed this Sep 14, 2024
@Cyberboss Cyberboss deleted the patch-1 branch September 14, 2024 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📚 documentation This issue is about working on our documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants