Update example.runtime.properties

[awoods]

See #194 for details

Resolves: https://jira.lyrasis.org/browse/VIVO-1930

[awoods]

Supersedes: #214

[brianjlowe]

Looks great. My only question is whether it would be more consistent to comment out the (empty) SMTP/replyTo properties rather than leaving them uncommented with empty values.

[gneissone]

Looks great. My only question is whether it would be more consistent to comment out the (empty) SMTP/replyTo properties rather than leaving them uncommented with empty values.

If I recall correctly VIVO complains if the properties are missing. Let's see if my memory is correct...

Edit - must have been a fever dream, seems to work fine.

[gneissone]

@awoods Just occurred to me we should update this value, too

VIVO/home/src/main/resources/config/default.runtime.properties

Line 191 in 52139be

# languages.selectableLocales = en_US, es_GO

es_GO doesn't exist for VIVO anymore, correct? I imagine the logic for using the non-standard locale is still in the source, but the language files have been renamed to just 'es' if I recall correctly.

[awoods]

@awoods Just occurred to me we should update this value, too

VIVO/home/src/main/resources/config/default.runtime.properties

Line 191 in 52139be

# languages.selectableLocales = en_US, es_GO

es_GO doesn't exist for VIVO anymore, correct? I imagine the logic for using the non-standard locale is still in the source, but the language files have been renamed to just 'es' if I recall correctly.

Since that will require additional testing, let's keep the scope of this set of PRs constrained... and revisit es_GO as a follow-on activity.

[awoods]

@gneissone : since both you and I have fingerprints on this PR, it would be nice if someone else performed the "squash/merge".