search

chenejac / VIVOTestMigration

0 stars 0 forks source link

VIVO-1982: Add example email settings, but do not populate #1865

Closed chenejac closed 3 years ago

chenejac commented 3 years ago

Benjamin Gross (Migrated from VIVO-1982) said:

Re-add email default info and logic from here: [https://github.com/vivo-project/VIVO/pull/214/files]

 

It was removed in the reversion of the deployment strategy. 

 

Setting the default email settings to blank values removes the smoke test warning about email parameters not being set on initial startup.

chenejac commented 3 years ago

Benjamin Gross said:

See [https://github.com/vivo-project/Vitro/pull/229] and https://github.com/vivo-project/VIVO/pull/238

chenejac commented 3 years ago

Brian Lowe said:

Resolved in commits [https://github.com/vivo-project/VIVO/commit/5d257cae6bace39af29278dadaf2015a6ca8ee45] and https://github.com/vivo-project/Vitro/commit/5075d7853775c00334991a2257795d89f9c324c3