Change the setting in sample new event

[mattstratton]

This allows for use of the default square image as an override. Open to bikeshedding as if it should be defaulting to "true" on a new event.

[bridgetkromhout]

Can we put the logic elsewhere? Make the config file have the setting start out empty. If not set, then use default. Prompt for the file when creating event but if it's not set, leave empty and use default.

I don't think it's helpful for us to be adding settings that are basically that logic flow?

[mattstratton]

Oh, that makes a lot of sense. If it's not set, just show the default. I honestly think that's what the code does right now anyway.

Yep. Just checked. That's what it does:

                    {{- if .event_logo_square }}
                      {{- if ne .event_logo_square "" }}
                        <img src = "/events/{{ .name }}/{{ .event_logo_square }}" class="img-fluid" alt="DevOpsDays {{ .city }}"/>

                      {{- end -}}
                      {{- else -}}
                        <img src="/img/event-logo-default.png" class="img-fluid" alt="DevOpsDays {{ .city }}"/>
                    {{- end -}}

I suppose it already works as designed, then. There's no need for a setting; you just HAVE to name the file that.

[mattstratton]

Hmm. We just need to tell them that not only do they need to add that setting, but the file has to be called logo-square.jpg.

It seems like a weird setting to make be the filename, since that implies it can be something non-boolean.

[mattstratton]

We can also go back to the "check if there is a logo-square.jpg in the event's directory" logic, which is starting to seem like what we should do.

[bridgetkromhout]

We can also go back to the "check if there is a logo-square.jpg in the event's directory" logic, which is starting to seem like what we should do.

Fine by me. Point being, if it's not actually configurable, making it a configuration setting seems off to me.

[bridgetkromhout]

If I recall correctly, checking in the yaml was easier than inspecting the filesystem. I'm fine with either.

[mattstratton]

It's configureable, just not configurable as to what it is. It's default/custom :)

[mattstratton]

Closing and moving the 🚲 🏠 to #255