[bug]: Change Cover button is missing on self-hosted version

[mikhail-shevtsov-wiregate]

Is there an existing issue for this?

• I have searched the existing issues

Current behavior

On latest release v0.7.1 when creating new project or editing project Change Cover button is missing on self-hosted version.

Steps to reproduce

1. Go to Projects;
2. Click Add Project button;
   or
3. Go to Project Settings;

In both cases the button Change Cover is Missing

Browser

Mozilla Firefox

Version

Self-hosted

[ChandanJal]

Yes, mine also not showing

[rhea0110]

Hey @mikhail-shevtsov-wiregate, thank you for reporting this bug. Our engineers have already started working on fixing this issue, and it will be live in some time. We will keep you updated on the progress made.

Thank you!

[ix64]

I found the reason.

The environment variables read via process.env.NEXT_PUBLIC_UNSPLASH_ENABLED are build-time variables, not run-time.

plane/web/components/core/image-picker-popover.tsx

Lines 24 to 26 in 61672f4

	const unsplashEnabled =
	process.env.NEXT_PUBLIC_UNSPLASH_ENABLED === "true" ||
	process.env.NEXT_PUBLIC_UNSPLASH_ENABLED === "1";

---

I suggested that use Next Runtime Config¹ to pass it.

By using this, we will not need to use this script to change NEXT_PUBLIC_API_BASE_URL any more.

plane/replace-env-vars.sh

Lines 5 to 15 in 61672f4

	if [ "${FROM}" = "${TO}" ]; then
	echo "Nothing to replace, the value is already set to ${TO}."
	exit 0
	fi
	# Only perform action if $FROM and $TO are different.
	echo "Replacing all statically built instances of $FROM with this string $TO ."
	grep -R -la "${FROM}" $DIRECTORY/.next | xargs -I{} sed -i "s|$FROM|$TO|g" "{}"

I'm willing to submit a PR to implement it.

Footnotes

1. https://nextjs.org/docs/pages/api-reference/next-config-js/runtime-configuration ↩

[vihar]

thanks for flagging this, @mikhail-shevtsov-wiregate. this issue is fixed on #2375.