Skip to content

Conversation

@sgryt
Copy link
Contributor

@sgryt sgryt commented Oct 8, 2025

@sgryt sgryt requested review from fkj and kasperhj October 8, 2025 10:41
@netlify
Copy link

netlify bot commented Oct 8, 2025

Deploy Preview for criipto-docs ready!

Name Link
🔨 Latest commit eb4f741
🔍 Latest deploy log https://app.netlify.com/projects/criipto-docs/deploys/68e63fe47bb39a0008276dfc
😎 Deploy Preview https://deploy-preview-225--criipto-docs.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 project configuration.

Copy link
Collaborator

@fkj fkj left a comment

Choose a reason for hiding this comment

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

I'm not sure this is correct? From a quick code inspection, FTN should also support using the Accept-Language header.

@sgryt
Copy link
Contributor Author

sgryt commented Oct 8, 2025

I'm not sure this is correct? From a quick code inspection, FTN should also support using the Accept-Language header.

Could be - but seems to be what the reported problem suggests. I have not tried reproducing the issue, though (have no immediate tooling available for experimenting with signed requests)

@sgryt
Copy link
Contributor Author

sgryt commented Oct 8, 2025

I'm not sure this is correct? From a quick code inspection, FTN should also support using the Accept-Language header.

Could be - but seems to be what the reported problem suggests. I have not tried reproducing the issue, though (have no immediate tooling available for experimenting with signed requests)

I agree that a quick code inspection indicates that this should still be supported (Accept-Language) - will try to repro tonight.

@sgryt sgryt marked this pull request as draft October 8, 2025 11:45
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.

3 participants