Skip to content
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

Revert "chore(location): Accept URL-like object" #10473

Merged
merged 1 commit into from
Apr 18, 2024

Conversation

dac09
Copy link
Collaborator

@dac09 dac09 commented Apr 18, 2024

Reverts #10467

We need the URL-like object because with server rendering, and SSR - we should let people have access to more properties like origin.

Motivation here described in the original PR here: #10441

We need origin for example, for generating the ogImage url in hooks

@dac09 dac09 requested a review from Tobbe April 18, 2024 07:34
@dac09 dac09 added the release:chore This PR is a chore (means nothing for users) label Apr 18, 2024
@dac09 dac09 added this to the RSC milestone Apr 18, 2024
@dac09 dac09 added the changesets-ok Override the changesets check label Apr 18, 2024
@dac09 dac09 enabled auto-merge (squash) April 18, 2024 07:43
@dac09 dac09 merged commit 78ed965 into main Apr 18, 2024
49 of 59 checks passed
@dac09 dac09 deleted the revert-10467-tobbe-location-url-path branch April 18, 2024 07:49
dac09 added a commit to dac09/redwood that referenced this pull request Apr 18, 2024
* 'main' of github.com:redwoodjs/redwood:
  Revert "chore(location): Accept URL-like object" (redwoodjs#10473)
  RSC: Be consistent about inlining rollup input (redwoodjs#10472)
  chore(paths): Remove outdated comment (redwoodjs#10471)
  feat(server-auth): Update getAuthenticationContext to support cookies and tokens both (redwoodjs#10465)
  chore(location): Accept URL-like object (redwoodjs#10467)
  fix(router): Remove barrel exports from router.tsx (redwoodjs#10464)
  chore(dbauth-mw): Refactor web side dbAuth creation (redwoodjs#10460)
  chore(router): Prevent circular dependency for namedRoutes (redwoodjs#10463)
  chore(router): route-validators: Better types and clean up comments (redwoodjs#10462)
  feat(server-auth): dbAuth 3/3 -  handle login, logout, signup, etc. requests if forwarded from middleware (redwoodjs#10457)
  docs(router): Document new NavLink className replacement behavior (redwoodjs#10401)
  chore(refactor): Split the router out into smaller logical units (redwoodjs#10434)
  feat(server-auth): Part 1/3: dbAuth middleware support (web side changes) (redwoodjs#10444)
  chore(auth): Build: Put ESM at the root, and CJS in /cjs (redwoodjs#10458)
  fix(ssr): Successfully serve static assets like `favicon.png` (redwoodjs#10455)
  chore(deps): update chore (redwoodjs#10367)
  (docs) Fix useCache headers and links (redwoodjs#10451)
  chore: remove aws-lambda (redwoodjs#10450)
  chore(deps): update dependency typescript to v5.4.5 (redwoodjs#10452)
@Josh-Walker-GM Josh-Walker-GM modified the milestones: RSC, v8.0.0 Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changesets-ok Override the changesets check release:chore This PR is a chore (means nothing for users)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants