-
Notifications
You must be signed in to change notification settings - Fork 1
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
Implement library for custom scaffolder actions. #59
Conversation
Xantier
commented
Feb 20, 2024
•
edited
Loading
edited
- Add functionality to receive custom scaffolder action triggering
- Add context provider to handle action running
- Add workspace manager to retrieve and upload wanted scaffolder workspace from/to S3 based on provided presigned URLs
- Add CustomScaffolderAction class to wrap the needed endpoint calls to create a simpler library interface
- Add docs
This pull request has been linked to Shortcut Story #18723: Spike: Create epic for Custom Scaffolder Actions. |
} | ||
|
||
|
||
async log(content: string, context?: Record<string, string>) { |
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.
Simple custom scaffolder context which exposes a logger, workspace and the payload passed into when the action was triggered
Add docs
src/lib/entityProvider/constants.ts
Outdated
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.
entityProvider side of the agent is untouched, just moved to a folder
); | ||
|
||
logger.info('File uploaded'); | ||
const etag = (response?.headers?.etag as string)?.replaceAll('"', ''); |
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.
AWS S3 produces etags that are wrapped in "
. It's implemented according to spec and will never change
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.
can't see any problems