quepid icon indicating copy to clipboard operation
quepid copied to clipboard

Fix issue when T&C URL is null

Open jzonthemtn opened this issue 2 years ago • 2 comments

Describe the bug When a value for the TC_URL is not set, there will be an error.

To Reproduce Don't set a value for the TC_URL environment variable. Look in the code for an error about size.

Expected behavior The code should not display a Terms and Conditions link if TC_URL is not set.

jzonthemtn avatar Oct 17 '22 15:10 jzonthemtn

<% if Rails.application.config.terms_and_conditions_url.size > 0 %>

Should probably change .size to something that is null-safe.

jzonthemtn avatar Oct 17 '22 15:10 jzonthemtn

This issue may affect other environment variables so might a good idea to check the use of others, too.

jzonthemtn avatar Oct 17 '22 15:10 jzonthemtn

I took a pass on the other urls to make sure okay, and they were!

epugh avatar Oct 27 '22 16:10 epugh