Skip to content

Conversation

@hunterachieng
Copy link
Contributor

@hunterachieng hunterachieng commented Sep 1, 2025

Summary

Add core or community labels to adaptors.

Fixes #584

Details

Add a core or community label to adaptor names. i.e `zoho (Core)

AI Usage

Please disclose how you've used AI in this work (it's cool, we just want to
know!):

  • Code generation (copilot but not intellisense)
  • Learning or fact checking
  • Strategy / design
  • Optimisation / refactoring
  • Translation / spellchecking / doc gen
  • Other
  • I have not used AI

You can read more details in our
Responsible AI Policy

Review Checklist

Before merging, the reviewer should check the following items:

  • Does the PR do what it claims to do?
  • If this is a new adaptor, added the adaptor on marketing website ?
  • If this PR includes breaking changes, do we need to update any jobs in
    production? Is it safe to release?
  • Are there any unit tests?
  • Is there a changeset associated with this PR? Should there be? Note that
    dev only changes don't need a changeset.
  • Have you ticked a box under AI Usage?

Signed-off-by: Hunter Achieng <achienghunter@gmail.com>
Signed-off-by: Hunter Achieng <achienghunter@gmail.com>
Signed-off-by: Hunter Achieng <achienghunter@gmail.com>
…cs-lable

Signed-off-by: Hunter Achieng <achienghunter@gmail.com>
@hunterachieng hunterachieng marked this pull request as ready for review September 2, 2025 19:26
Signed-off-by: Hunter Achieng <achienghunter@gmail.com>
@hunterachieng
Copy link
Contributor Author

@josephjclark it is fixed

Copy link
Collaborator

@josephjclark josephjclark left a comment

Choose a reason for hiding this comment

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

Awesome - so this PR just says "If an adaptor has a badge, we'll add it to the docs.json and pass it to the doc site"

Happy to merge this as it is - no changeset needed - but I note that no badges are actually added yet. Would you like to set a couple so that you have something to work with on the docs side?

Signed-off-by: Hunter Achieng <achienghunter@gmail.com>
@hunterachieng
Copy link
Contributor Author

@josephjclark I have added a few badges to test with for the docs

@josephjclark josephjclark merged commit 2dff5cd into main Sep 11, 2025
2 checks passed
@josephjclark josephjclark deleted the feature/584-docs-lable branch September 11, 2025 12:38
@jackohilts
Copy link

What does this end up impacting now that it's merged? I can't see any badges on the docs site.

Started a doc here to discuss this a bit more - I don't feel great about the impact of this on the Adaptor Ecosystem as a whole just yet, and I think we should do some design/product thinking on it first: https://docs.google.com/document/d/1UKBZWczNXE2gez14nwqsTy-FF1RIkNFLOw0peUp1vqQ/edit?usp=sharing

@hunterachieng
Copy link
Contributor Author

@jackohilts This does not affect anything. The badges will only be seen after we merge the docs PR which handles the display of the badge properties.

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.

Set up the concept of a Core adaptor

4 participants