fix(send-and-archive): Don't validate defaultSendAction key in config (#3031)

Before this commit, opening a new composer window would reset the
default send action to just `send`, even if you had selected it to be
`send-and-archive` via preferences.

This happened because on window initialization we run a config schema
validation, and coerce any values that are not valid under the schema
(e.g. make sure that the value for a setting is one of possible values specified
in the enum). Given that the schema is static, but the values for the enum for
this particular setting are dynamically calculated when we render the selector,
it doesn't make sense to validate the values for this setting.

To prevent validation, we just need to remove the `enum`, and `enumLabels`
from the static config-schema.
This commit is contained in:
Juan Tejada 2016-11-09 11:28:46 -08:00
parent 5221a87947
commit 628a4b8617
2 changed files with 1 additions and 3 deletions

View file

@ -124,8 +124,6 @@ export default {
defaultSendType: {
'type': 'string',
'default': 'send',
'enum': ['send'],
'enumLabels': ['Send'],
'title': "Default send behavior",
},
undoSend: {

@ -1 +1 @@
Subproject commit bb3d8d0a6bf102e64a47d2017f3225c11ec9e7cd
Subproject commit 547d21bc96561fdd09087cf5ebd2fad1fd779e6c