-
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
Roughs in initial strategies #1
Conversation
To be clear: the goal of this library is to provide a single API that supports generating on-demand resized image URLs from a variety of services. Everyone handles this differently and it's annoying to get right sometimes (Imgix's MD5 signature or the Lambda's Base64 encoding and payload format). |
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.
Looks great! 🔥
const query = stringify(params, { | ||
arrayFormat: "brackets", | ||
skipNulls: true, | ||
sort: (a, b) => a.localeCompare(b), |
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.
Nice
|
||
const params = { | ||
bucket: source.bucket, | ||
key: src.replace(`${source.source}/`, ""), |
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.
Nice
660a15a
to
ef2ffb2
Compare
210d388
to
411d1f8
Compare
Preliminary local integration looks good. Going to merge and get CI set up; will integrate locally replacing the underlying Gemini URL creation then feature flag a service switcher and pass it along as a header to Metaphysics to sync the chosen service. |
🚀 PR was released in |
Will publish this locally using Yalc to drive out how to do integration in Force/Metaphysics along with setting a desired service.