Ticket #10772 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Add option in first-usage screen to turn of usage statistics

Reported by: Peter Peterson Owned by: Nick Draper
Priority: blocker Milestone: Release 3.3
Component: Framework Keywords:
Cc: Blocked By: #10559, #10767
Blocking: Tester: Peter Peterson

Description


Change History

comment:1 Changed 6 years ago by Nick Draper

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Nick Draper

  • Blocked By 10767 added

comment:3 Changed 6 years ago by Nick Draper

  • Status changed from assigned to inprogress

re #10772 add opt out with confirmation to first use dialog

Changeset: 861f00818cb7c768bbbaa582f155ddfb2ad9ac69

comment:4 Changed 6 years ago by Nick Draper

re #10772 try to decrease conflicts

Changeset: 96a26db3f16a99decbe6c44cf81c2f8253435519

comment:5 Changed 6 years ago by Nick Draper

re #10772 try to decrease conflicts

Changeset: 656e5a35fbfa7531d98d76e52212506f9821b821

comment:6 Changed 6 years ago by Nick Draper

re #10772 merge conflict resolution

Merge branch 'feature/10772_OptOut_usage_stats' into develop

Conflicts:

Code/Mantid/MantidPlot/src/Mantid/FirstTimeSetup.ui

Changeset: 07fcbd27e189e983622753e1da837c44c2328163

comment:7 Changed 6 years ago by Nick Draper

  • Status changed from inprogress to verify
  • Resolution set to fixed

To test

  1. Startup Mantidplot
  2. in the first time setup ui (help->first time setup), there should now be a box for Report usage data.
  3. Check it strongly suggests you should leave it on when trying to turn it off.
  4. Check the value is saved, and reloaded properly.
  5. Check this has not screwed up the display of the dialog.

For testing and merging - you are likely to run into merging problems for the UI file FirstTimeSetup.ui.

The best approach to resolution for this is to take the version in the branch for all changes. I've already merged in master and all other branches that could alter this file.

comment:8 Changed 6 years ago by Nick Draper

merge in master to solve ui conflict re #10772

Changeset: 178f3f67b1621025f52ce8f2e3a4893c6bea8e32

comment:9 Changed 6 years ago by Peter Peterson

  • Status changed from verify to verifying
  • Tester set to Peter Peterson

comment:10 Changed 6 years ago by Nick Draper

  • Status changed from verifying to closed

Merge branch 'feature/10767_improve_linux_command_link_buttons' of github.com:mantidproject/mantid into feature/10772_OptOut_usage_stats

Full changeset: 71f4af60c9f2365743587a0acbaeb7b7d0787724

comment:11 Changed 6 years ago by Nick Draper

Merge branch 'master' of github.com:mantidproject/mantid into feature/10772_OptOut_usage_stats

Full changeset: d2f9c0d3180accfcd5eaf494c3905b77648109af

comment:12 Changed 6 years ago by Pete Peterson

Merge remote-tracking branch 'origin/feature/10772_OptOut_usage_stats'

Full changeset: a826649797ea37bf70aa8a1aef238ed6be442ad2

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11614

Note: See TracTickets for help on using tickets.