-
Notifications
You must be signed in to change notification settings - Fork 7.3k
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
destination
does not start with /
, http://
, or https://
for route
#14310
Comments
Hey @olivierlambert! 👋 It's Dosu again. After reviewing the error log you've provided, it seems like the issue is related to an invalid rewrite rule in your configuration. The error message This usually happens when environment variables or configuration settings that are supposed to define these URLs are not correctly set, leading to the placeholder Here's a step-by-step guide to troubleshoot and potentially fix this issue:
If after following these steps the issue remains unresolved, it might be helpful to provide more details about your configuration and the specific setup you're using. This could include parts of your I hope this helps get you closer to a solution! If you have any more questions or need further assistance, feel free to ask. 🚀
|
I didn't not change the .env file and I never had this issue before, so it might worth it to have an actual human taking a look at some point :D I'm taking a deeper look on what I could have done wrong, but I found nothing right now. |
Hey @olivierlambert! I understand the frustration when things don't work as expected, especially when you haven't made any changes that would obviously cause the issue. While I've tried to provide some guidance, it seems like this might require a closer look from someone with more context on the project's specifics. @keithwillcode, this one's a bit out of my wheelhouse, could you step in and provide your guidance? In the meantime, @olivierlambert, if there's anything else you think might be relevant or if you've made any discoveries on your end, please feel free to share. Your insights could be very helpful in diagnosing the issue more quickly.
|
So it seems there's a new var in |
Adding the var in |
@olivierlambert it's used here cal.com/apps/web/next.config.js Line 309 in 0ee81d0
|
I'll take a look at this |
@olivierlambert glad you found the new .env. we are moving to a new API right now and once thats done we are introducing proper upgrade paths starting at cal.com v5.0 with in-depth upgrading tutorials and scripts with https://codemod.com 🥳 |
Understood. As long as the env file explains the new additions and what should be used (or a changelog), I am perfectly happy with it! |
Issue Summary
Updating from v3.8.5 to v3.9.4 leads to an error during
yarn build
Steps to Reproduce
Actual Results
Expected Results
Build would work
Technical details
The text was updated successfully, but these errors were encountered: