Skip to content

Conversation

@vladimirvolek
Copy link
Member

@vladimirvolek vladimirvolek commented Mar 11, 2025

resolves: #230

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Mar 11, 2025

Deploying blockfrost-platform with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6b73561
Status: ✅  Deploy successful!
Preview URL: https://512bcd73.blockfrost-platform.pages.dev
Branch Preview URL: https://blockfrost-response-header.blockfrost-platform.pages.dev

View logs

@vladimirvolek vladimirvolek marked this pull request as ready for review March 11, 2025 21:56
Copy link
Collaborator

@ginnun ginnun left a comment

Choose a reason for hiding this comment

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

lgtm

@ginnun ginnun merged commit c84a123 into main Mar 12, 2025
10 checks passed
@michalrus michalrus mentioned this pull request Mar 17, 2025
// insert a static header value. Using unwrap is safe here because the literal is known to be a valid header.
response_headers.insert(
"blockfrost-platform-response",
"HeaderValue".parse().unwrap(),
Copy link
Member

Choose a reason for hiding this comment

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

It’s only a static value. 🙃 I reopened the original issue:

@vladimirvolek said he’ll fix it.

@michalrus michalrus deleted the blockfrost-response-header branch March 17, 2025 15:55
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.

Tx hash in the header

4 participants