Skip to content

Conversation

@kieranlangton
Copy link

@kieranlangton kieranlangton commented Jan 23, 2026

📋 Summary

Renaming some folders and properties to conform to the new schema


🔗 Related issue(s)


🧩 Plugin details

  • Plugin name:
  • Type of change:
    • Bug fix
    • New datastream
    • Enhancement to existing datastream
    • Performance improvement
    • Documentation / metadata / logo
    • Other (please describe):

⚠️ Breaking changes

Does this PR introduce any breaking changes?

  • No
  • Yes (please describe):

If yes, describe:

  • What breaks
  • Who is impacted
  • Any migration steps

📚 Documentation

  • Documentation updated
  • No documentation changes needed

✅ Checklist

  • No secrets or credentials included
  • Plugin, datastream and UI naming follow SquaredUp guidelines
  • I agree to the Code of Conduct

Copilot AI review requested due to automatic review settings January 23, 2026 15:00
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the DigiCert plugin to conform to a new schema version 2.0 by restructuring the configuration properties in the metadata and datastream files.

Changes:

  • Restructured base plugin configuration to use nested base object
  • Renamed datastream configuration properties from dataSourceConfig to config and dataSourceName to baseDataSourceName

Reviewed changes

Copilot reviewed 3 out of 6 changed files in this pull request and generated no comments.

File Description
plugins/DigiCert/v1/metadata.json Restructured base plugin configuration into nested base object with plugin, majorVersion, and config properties
plugins/DigiCert/v1/data_streams/certificateOrders.json Renamed dataSourceConfig to config and dataSourceName to baseDataSourceName
plugins/DigiCert/v1/data_streams/accountUsers.json Renamed dataSourceConfig to config and dataSourceName to baseDataSourceName

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants