Hello,
I believe there are no issues left that I'm aware of. However, as you've
said, tdf files must be backed up manually before reinstallation or
uninstallation at this time, otherwise they will be lost permanently.
The next update will have automatic silent backup procedure, at least
for reinstallation cause I think it's not needed for uninstallation. If
people uninstall something, it's expected that they will not use it
anymore, so in that case backup is not needed.
"Joseph Lee" <joseph.lee22590@xxxxxxxxx> wrote:
Hi,
All review materials (license, security, user experience) passes.
A possible suggestion for 2.0: it should be possible for the add-on installer
to keep the old tdf files when installing new add-on releases (this involves
creating installTasks.py).
If you believe it is ready for stable branch, then I'll go ahead and make it
available under the stable tag for now (I'm facing difficulty in trying to make
it work under translations workflow at this point due to redesign of Bitbucket).
Cheers,
Joseph
-----Original Message-----
From: nvda-addons-bounce@xxxxxxxxxxxxx
[mailto:nvda-addons-bounce@xxxxxxxxxxxxx] On Behalf Of Hrvoje Katic
Sent: Wednesday, July 20, 2016 2:45 AM
To: nvda-addons@xxxxxxxxxxxxx
Subject: [nvda-addons] Re: Tone master 1.1
A major issue has now been fixed in 1.2. Thanks, Josef!
https://github.com/nvdaaddons/toneMaster/releases/download/v1.2/toneMaster-1.2.nvda-addon
"Joseph Lee" <joseph.lee22590@xxxxxxxxx> wrote:
Hi,
Review results:
* Licensing: pass (carried from previous version)
* Security: pass (thanks for addressing some issues).
* User experience: pass with caution.
The way it was done was to review the diff between 1.0 and 1.1.
Comments:
* Possible bug found: if you create an empty tone data file, select
it, attempt to play it, then select another one, NVDA will not play the newly
chosen tone data (will file a bug report on GitHub).
I recommend a green light once major bugs are resolved. Thanks. Reviews form
others are appreciated.
Cheers,
Joseph
----------------------------------------------------------------
NVDA add-ons: A list to discuss add-on code enhancements and for reporting
bugs.
Community addons are available from: http://addons.nvda-project.org To send a ;
message to the list: nvda-addons@xxxxxxxxxxxxx To change your list
settings/unsubscribe: //www.freelists.org/list/nvda-addons
To contact list moderators: nvda-addons-moderators@xxxxxxxxxxxxx
----------------------------------------------------------------
NVDA add-ons: A list to discuss add-on code enhancements and for reporting
bugs.
Community addons are available from: http://addons.nvda-project.org
To send a message to the list: nvda-addons@xxxxxxxxxxxxx
To change your list settings/unsubscribe:
//www.freelists.org/list/nvda-addons
To contact list moderators: nvda-addons-moderators@xxxxxxxxxxxxx
----------------------------------------------------------------
NVDA add-ons: A list to discuss add-on code enhancements and for reporting
bugs.
Community addons are available from: http://addons.nvda-project.org
To send a message to the list: nvda-addons@xxxxxxxxxxxxx
To change your list settings/unsubscribe:
//www.freelists.org/list/nvda-addons
To contact list moderators: nvda-addons-moderators@xxxxxxxxxxxxx
