Skip to content

Conversation

eduardoboucas
Copy link
Member

Which problem is this pull request solving?

Adds a getMetadata() method that issues a HEAD request for a given key.

Closes https://github.com/netlify/pillar-runtime/issues/764.

@eduardoboucas eduardoboucas requested a review from a team as a code owner November 2, 2023 11:18
Copy link

netlify bot commented Nov 2, 2023

Deploy Preview for blobs-js ready!

Name Link
🔨 Latest commit fa6dd62
🔍 Latest deploy log https://app.netlify.com/sites/blobs-js/deploys/65438687aa2e2e000837ef7a
😎 Deploy Preview https://deploy-preview-90--blobs-js.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions github-actions bot added the type: feature code contributing to the implementation of a feature and/or user facing functionality label Nov 2, 2023
async getWithMetadata(
key: string,
options?: GetWithMetadataOptions,
): Promise<{ data: string } & GetWithMetadataResult>
): Promise<({ data: string } & GetWithMetadataResult) | null>
Copy link
Member Author

Choose a reason for hiding this comment

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

Not related with the new method, but I realised the types for getWithMetadata() were incorrect, since the method can return null and that wasn't being included in all the overloads.

@eduardoboucas eduardoboucas merged commit 0327476 into main Nov 3, 2023
@eduardoboucas eduardoboucas deleted the feat/getmetadata-method branch November 3, 2023 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature code contributing to the implementation of a feature and/or user facing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants