Bug 20106 - Hea - Add notes to system preferences (administration page, example for geolocation)
Summary: Hea - Add notes to system preferences (administration page, example for geolo...
Status: RESOLVED DUPLICATE of bug 24682
Alias: None
Product: Koha
Classification: Unclassified
Component: System Administration (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Jasmine Amohia
QA Contact: Testopia
URL:
Keywords: Academy
Depends on:
Blocks:
 
Reported: 2018-01-30 20:31 UTC by Katrin Fischer
Modified: 2020-04-19 01:47 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 20106: Hea - Added notes to sysprefs (administration page, example for geolocation) (4.24 KB, patch)
2019-01-17 02:09 UTC, Jasmine Amohia
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Katrin Fischer 2018-01-30 20:31:47 UTC
At the moment we have a nice administration page for Hea, but also have system preferences where you can set those values up for a single library.

We should:
- Add a note above the preferences to tell people about the administration page
- Add an example for UsageStatsGeolocation  to make the way to use it more clear.
Comment 1 Aleisha Amohia 2019-01-16 22:23:28 UTC
The geolocation is looking for longitude and latitude. A nice recognisable example might be:
Longitude of Eiffel Tower: 2.294481
Latitude of Eiffel Tower: 48.858370
Comment 2 Jasmine Amohia 2019-01-17 02:09:06 UTC
Created attachment 84114 [details] [review]
Bug 20106: Hea - Added notes to sysprefs (administration page, example for geolocation)

Test plan:
1) Go to system preferences and search for UsageStats.
2) Confirm that the system preferences all have a note with a link directing user to a page to
update their usage statistics
3) In system preferences for UsageStats, notice that the description of
sys pref 'UsageStatsGeolocation' provides an example with the Eiffel
Tower's geolocation.
Comment 3 David Nind 2019-01-18 01:50:35 UTC
Hi Jasmine.

I feel like I'm being a bit picky here, but the QA team would probably pick these things up!

If you could make these changes, I will sign it off:

- Put a space between NOTE: and the start of the sentence, for example NOTE: Whatever the note is. This makes it consistent with the other notes.

- Add a full stop at the end of the note sentence, for example NOTE: Whatever the note is. This also makes it consistent with the other notes.

- The note is missing for the UsageStatsLibraryName preference.

- Change the text for the note and link from 'Usage statistics can also be configured here' to something like 'Configure all the usage statistics settings.'.

(When writing for the web I usually try and avoid writing the equivalent of 'click here', and generally try to make link text name meaningful. It is also a web accessibility thing as well. The QA team/translation team might have some thoughts on what is better from a translation perspective.)

Thanks!

This is a really nice improvement.

David
Comment 4 Katrin Fischer 2019-01-18 12:26:46 UTC
> (When writing for the web I usually try and avoid writing the equivalent of
> 'click here', and generally try to make link text name meaningful. It is
> also a web accessibility thing as well. The QA team/translation team might
> have some thoughts on what is better from a translation perspective.)

<3 I thought, I was the only one :)
Comment 5 Katrin Fischer 2020-04-19 01:47:06 UTC
This appears to have been fixed with bug 24682, as we now have nice notes on the prefs:

NOTE: You must use the dedicated admin configuration page to edit this preference. 

Also the fields are locked, so you can't change them from the system preference editor.

*** This bug has been marked as a duplicate of bug 24682 ***