Project

General

Profile

Bug #17322

Update tooltip explaining "Map ACLs"

Added by Warren Block almost 3 years ago. Updated over 1 year ago.

Status:
Resolved
Priority:
Nice to have
Assignee:
William Grzybowski
Category:
GUI (new)
Target version:
Seen in:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
Yes
Needs Doc:
Yes
Needs Merging:
Yes
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

The new Mode option has been added to the AFP service (https://bugs.freenas.org/issues/16870). The tooltip does not describe what this or the other Map ACLs options do, but should. Suggested wording:

How to map effective permissions for authenticated users: _Rights_ (default, Unix-style permissions), _Mode_ (ACLs), or _None_

models.py (19 KB) models.py Warren Block, 09/06/2016 01:23 PM
models.py.diff (788 Bytes) models.py.diff Warren Block, 11/17/2016 09:41 AM

Associated revisions

Revision 3c8274c0 (diff)
Added by William Grzybowski almost 3 years ago

fix(gui): update several help texts in services models

Ticket: #17322

Revision fa6cfb32 (diff)
Added by William Grzybowski almost 3 years ago

fix(gui): typo

Ticket: #17322

Revision 5b6e5cce (diff)
Added by William Grzybowski almost 3 years ago

fix(gui): another typo!

Ticket: #17322

Revision 72e9fc75 (diff)
Added by William Grzybowski almost 3 years ago

fix(gui): typo in help text

Ticket: #17322

History

#1 Updated by Heather Ownby almost 3 years ago

  • Assignee set to Dru Lavigne

#2 Updated by Warren Block almost 3 years ago

  • Assignee deleted (Dru Lavigne)

Docs have already been updated, it is the tooltip in the GUI that needs it now. Thanks!

#3 Updated by Dru Lavigne almost 3 years ago

  • Assignee set to William Grzybowski

#4 Updated by William Grzybowski almost 3 years ago

  • Status changed from Unscreened to Screened
  • Target version set to 9.10.2

#5 Updated by Warren Block almost 3 years ago

This patch adds the suggested text to models.py and edits some of the other help strings for clarity or grammar.

#6 Updated by William Grzybowski almost 3 years ago

  • Status changed from Screened to 19

Fixes branches will only be created after #16870 is merged.

#7 Updated by Warren Block almost 3 years ago

The text is still a bit off. New diff, hopefully less buggy but untested, spaces, commas, capitalization fixed.

#8 Updated by Warren Block almost 3 years ago

  • Status changed from 19 to Unscreened

#9 Updated by William Grzybowski almost 3 years ago

  • Status changed from Unscreened to Needs Developer Review

#10 Updated by Josh Paetzel almost 3 years ago

  • Status changed from Needs Developer Review to Reviewed

#11 Updated by Vaibhav Chauhan over 2 years ago

  • Status changed from Reviewed to Ready For Release

#12 Updated by Dru Lavigne over 1 year ago

  • Status changed from Ready For Release to Resolved

Also available in: Atom PDF