Skip to content

Openapi security#1606

Merged
Naramsim merged 3 commits into
PokeAPI:masterfrom
FallenDeity:openapi-security
Jul 17, 2026
Merged

Openapi security#1606
Naramsim merged 3 commits into
PokeAPI:masterfrom
FallenDeity:openapi-security

Conversation

@FallenDeity

Copy link
Copy Markdown
Contributor

Change description

Remove the default security and security schema fields from the openapi schema generation, had auth whitelisted in settings since we dont perform any auth

ps: was thinking about picking up the astro site in development and noticed openapi docs had auth due the source being here

cc: @Naramsim

AI coding assistance disclosure

None

Contributor check list

  • I have written a description of the contribution and explained its motivation.
  • I have written tests for my code changes (if applicable).
  • I have read and understood the AI Assisted Contribution guidelines.
  • I will own this change in production, and I am prepared to fix any bugs caused by my code change.

@Naramsim

Naramsim commented Jul 17, 2026

Copy link
Copy Markdown
Member

Great job! It would be cool to have in the openapi schema all the descriptions that we have in the docs right now and try to use the openapi schema to generate them. Moving forward with the Astro poc that we have at https://pokeapi.github.io/pokeapi.co/

@Naramsim Naramsim added the no-deploy Instructs the updater-bot not to deploy this PR label Jul 17, 2026
@Naramsim
Naramsim merged commit cd0a7a9 into PokeAPI:master Jul 17, 2026
19 checks passed
@pokeapi-machine-user

Copy link
Copy Markdown

This Pull Request won't be deployed since the label no-deploy was found in its meta-data.

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

Labels

no-deploy Instructs the updater-bot not to deploy this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants