From 8a5c63495787e631762afa44f3bed183e563e56d Mon Sep 17 00:00:00 2001 From: Jesse Duffield Date: Sat, 29 Jun 2019 13:56:58 +1000 Subject: [PATCH] don't ask about the reporting config thing more than once --- pkg/gui/gui.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/pkg/gui/gui.go b/pkg/gui/gui.go index c2eb2c11..ec9e1056 100644 --- a/pkg/gui/gui.go +++ b/pkg/gui/gui.go @@ -205,12 +205,15 @@ func (gui *Gui) loadNewDirectory() error { func (gui *Gui) promptAnonymousReporting() error { return gui.createConfirmationPanel(gui.g, nil, gui.Tr.AnonymousReportingTitle, gui.Tr.AnonymousReportingPrompt, func(g *gocui.Gui, v *gocui.View) error { gui.waitForIntro.Done() + // setting the value here explicitly so that we don't re-request after coming back from a subprocess. The proper solution would be to reload the config but that's tricky because it's loaded at the very top level + gui.Config.UserConfig.Reporting = "on" return gui.Config.WriteToUserConfig(func(userConfig *config.UserConfig) error { userConfig.Reporting = "on" return nil }) }, func(g *gocui.Gui, v *gocui.View) error { gui.waitForIntro.Done() + gui.Config.UserConfig.Reporting = "off" return gui.Config.WriteToUserConfig(func(userConfig *config.UserConfig) error { userConfig.Reporting = "off" return nil