-
Notifications
You must be signed in to change notification settings - Fork 4k
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
source-coassemble contribution from topefolorunso #45669
base: master
Are you sure you want to change the base?
source-coassemble contribution from topefolorunso #45669
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
/format-fix
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@topefolorunso PRs have todo-lists now! Check off items you've done already.
If it does not apply any formatting changes, that usually means that you'd need to merge a fresh master in first.
Everything that is in looks good, but I would love to see trackings and screen type streams. Those are "available on request only", so I think this needs a docs entry that enabling them without having them enabled on the API side would result in errors.
/format-fix
|
Correct! Do you want me to add the doc now? I think that should come in once the streams are implemented in the connector. @natikgadzhi |
@natikgadzhi Please clarify this part "Click into the CI workflows that wait for a maintainer to run them, which should trigger CI runs" |
What
This PR adds a new connector for source Coassemble (source-coassemble).
Contributor-provided connector description:
Coassemble is an online training tool that connects people with the information they need - anytime, anyplace.
Details
Expand for list of spec properties and streams
Configuration
user_token
string
user_id
string
Streams
Reviewer checklist
/format-fix
to resolve any formatting errors