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

force-static broken for catch all routes #62537

Open
alexanderhorner opened this issue Feb 26, 2024 · 2 comments · May be fixed by #62757
Open

force-static broken for catch all routes #62537

alexanderhorner opened this issue Feb 26, 2024 · 2 comments · May be fixed by #62757
Labels
bug Issue was opened via the bug report template. linear: next Confirmed issue that is tracked by the Next.js team. Pages Router Related to Pages Router.

Comments

@alexanderhorner
Copy link

alexanderhorner commented Feb 26, 2024

Link to the code that reproduces this issue

CodeSandbox

To Reproduce

  1. Build the application npm run build
  2. Start the application npm run start
  3. Refresh the root page a few times

Current vs. Expected behavior

Current behaviour: The time changes every second

Expected behaviour: Just like can be observed on the sub directories, the page should be generated on the first visit and then statically served on every subsequent visit (time stays the same).

Provide environment information

codesandbox.io:

Operating System:
  Platform: linux
  Arch: x64
  Version: #1 SMP PREEMPT_DYNAMIC Sun Aug  6 20:05:33 UTC 2023
  Available memory (MB): 4102
  Available CPU cores: 2
Binaries:
  Node: 20.9.0
  npm: 9.8.1
  Yarn: 1.22.19
  pnpm: 8.10.2
Relevant Packages:
  next: 14.1.1-canary.72 // There is a newer canary version (14.1.1-canary.73) available, please upgrade! 
  eslint-config-next: N/A
  react: 18.2.0
  react-dom: 18.2.0
  typescript: 5.1.3
Next.js Config:
  output: N/A

Which area(s) are affected? (Select all that apply)

App Router, Data fetching (gS(S)P, getInitialProps)

Which stage(s) are affected? (Select all that apply)

next build (local), next start (local)

Additional context

No response

NEXT-2622

@alexanderhorner alexanderhorner added the bug Issue was opened via the bug report template. label Feb 26, 2024
@github-actions github-actions bot added the Pages Router Related to Pages Router. label Feb 26, 2024
@balazsorban44 balazsorban44 added the linear: next Confirmed issue that is tracked by the Next.js team. label Feb 27, 2024
@coffeecupjapan
Copy link
Contributor

@alexanderhorner
I opened a PR for this issue.

I think I fix this issue. However, maybe there are lacks of consideration for some edge cases, and therefore should investigate for this issue more.

#62757

@alexanderhorner
Copy link
Author

Thank you. I agree it should be investigated further. I've also noticed some weird behavior with dynamicParams, wich doesn't seem to be consistent with the documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue was opened via the bug report template. linear: next Confirmed issue that is tracked by the Next.js team. Pages Router Related to Pages Router.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants