Skip to content

Conversation

@simonLeary42
Copy link
Contributor

@simonLeary42 simonLeary42 commented Oct 10, 2025

I see two ways of doing the same thing:

context["FOOBAR"] = import_from_settings("FOOBAR") -> {{ FOOBAR }}

versus:

SETTINGS_EXPORT.append("FOOBAR") -> {{ settings.FOOBAR }}

I assume that SETTINGS_EXPORT is preferred. Doing it the same way everywhere is a good thing, I think.

@aebruno
Copy link
Member

aebruno commented Oct 25, 2025

Thanks for cleaning this up!

@aebruno aebruno merged commit 46d0eba into coldfront:main Oct 25, 2025
1 check passed
@simonLeary42
Copy link
Contributor Author

Found a third way: {{ settings_value 'FOOBAR' }}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants