Replies: 4 comments
-
@huozhi @shuding @timneutkens from commit logs it looks like you three are the most active folks working on metadata who have approval / merge access -- if one of you can give me a sign-off on accepting this feature I'll happily put in a PR to implement it! |
Beta Was this translation helpful? Give feedback.
0 replies
-
+1 for this as well! |
Beta Was this translation helpful? Give feedback.
0 replies
-
+1 here 🥺 |
Beta Was this translation helpful? Give feedback.
0 replies
-
+1 need this! |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Goals
Allow syntax like this:
With this syntax,
es
would then expand tohttp://example.com/es/the/current/pathname
.Non-Goals
No response
Background
This would enable configuring alternates in a layout without having to propagate the configuration to each page file and manually expand pathnames. Without such syntax it's nearly impossible to reliably configure alternates across a large app since doing so would require that developers remember to inject the metadata on every page they create which is super error prone.
Proposal
In
next.js/packages/next/src/lib/metadata/resolvers/resolve-url.ts
Lines 75 to 80 in e838f26
And yes I'm happy to contribute if this contribution would be accepted.
Beta Was this translation helpful? Give feedback.
All reactions