Skip to content

Added CPO suggestion for plugin list & footnote with links to old documentation #PG-5194 & #PG-5185#956

Merged
AltamashShaikh merged 4 commits into
livefrom
PG-5184-add-descriptions
May 20, 2026
Merged

Added CPO suggestion for plugin list & footnote with links to old documentation #PG-5194 & #PG-5185#956
AltamashShaikh merged 4 commits into
livefrom
PG-5184-add-descriptions

Conversation

@lachiebol
Copy link
Copy Markdown
Contributor

@lachiebol lachiebol commented May 15, 2026

Description

fetch-openapi-specs.php will now also fetch and store metadata about our plugins that we can use.

Should still work without new endpoint, as the plugin names still use the old endpoint, but descriptions wont be available.

Checklist

  • [✔] I have understood, reviewed, and tested all AI outputs before use
  • [✔] All AI instructions respect security, IP, and privacy rules

Review

Comment thread app/templates/api-swagger-index.twig Outdated
Copy link
Copy Markdown
Contributor

@AltamashShaikh AltamashShaikh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

left 1 small fix comment and check this issue

app/templates/api-swagger-index.twig:13 always renders the colon separator even when pluginSpec.summary is empty. Since app/helpers/OpenApiSpecRegistry.php:148 intentionally returns no
  metadata when the JSON file is absent, the deployed page can show rows like Plugin Name: with no description. This is likely in the current branch unless the deploy process runs fetch-
  openapi-specs.php and publishes plugin_metadata_v1.0.0.json.

lachiebol and others added 2 commits May 18, 2026 09:32
@lachiebol lachiebol requested a review from AltamashShaikh May 17, 2026 21:40
@AltamashShaikh AltamashShaikh merged commit eb491d8 into live May 20, 2026
1 check passed
@AltamashShaikh AltamashShaikh deleted the PG-5184-add-descriptions branch May 20, 2026 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants