Dead link
Closed this issue · 18 comments
Hi, can you please share your environment with us and version?
I've tested in latest version and link seems to be working fine.
Hi,
it's selfhosted v14.04.2024.1.
@Civolilah can you replicate?
@Civolilah can you replicate?
@beganovich It works fine for me as well. The link for this element is actually pulled from the VITE_WHITELABEL_INVOICE_URL
environment variable, so missing the link in the .env file can cause this issue. @LarsK1, can you please confirm whether your .env file is populated with the correct value for the VITE_WHITELABEL_INVOICE_URL
variable?
@Civolilah no, there's no such variable in there. There also no such one in the .env.example.
To what value should I set it?
@Civolilah no, there's no such variable in there. There also no such one in the .env.example.
To what value should I set it?
@LarsK1 The variable was probably added to the .env.example file later, which is why you're missing it. Please ensure that you have
VITE_WHITELABEL_INVOICE_URL=https://app.invoiceninja.com/buy_now/?account_key=AsFmBAeLXF0IKf7tmi0eiyZfmWW9hxMT&product_id=3
in the .env file, and it will work for you. Thanks!
@Civolilah just added it, and also run php artisan optimize. Still no luck
The variable doesn't exist here as well: https://github.com/invoiceninja/invoiceninja/blob/v5-develop/.env.example
@LarsK1 Ah, there seems to be a slight misunderstanding. The variable needs to be added to the React/frontend project. Here is the current .env.example file, which is the best reference: https://github.com/invoiceninja/ui/blob/main/.env.example. Thanks!
You'll have to share with us how you get v5 working. Is it a Docker install, prebuilt ZIP or you did do everything yourself?
It's the prebuilt zip
Thanks.
cc @turbo124, might be able to help us :)
i think i know the issue, the link has changed:
https://invoiceninja.invoicing.co/client/subscriptions/O5xe7Rwd7r/purchase
Can we update the build for this url please.
Hi @turbo124 @Civolilah @beganovich ,
the big still exists, even though the pr is merged?