notes: default to internal, require --external flag
This commit is contained in:
@@ -176,10 +176,10 @@ def cmd_notes_list(args: argparse.Namespace) -> int:
|
||||
def cmd_notes_add(args: argparse.Namespace) -> int:
|
||||
svc = _service()
|
||||
def _op():
|
||||
result = svc.add_note(args.ticket_id, message=args.message, type_id=args.type_id, internal=args.internal, status_id=args.status_id, note_date=args.note_date)
|
||||
result = svc.add_note(args.ticket_id, message=args.message, type_id=args.type_id, internal=not args.external, status_id=args.status_id, note_date=args.note_date)
|
||||
data = result.get("Data") or result.get("result") or result
|
||||
return result, {"created_note_id": data.get("Id") or data.get("id"), "ticket_id": args.ticket_id}
|
||||
_print(_run_write("notes.add", {"ticket_id": args.ticket_id, "message": args.message, "type_id": args.type_id, "internal": args.internal, "status_id": args.status_id, "note_date": args.note_date}, _op))
|
||||
_print(_run_write("notes.add", {"ticket_id": args.ticket_id, "message": args.message, "type_id": args.type_id, "internal": not args.external, "status_id": args.status_id, "note_date": args.note_date}, _op))
|
||||
return 0
|
||||
|
||||
|
||||
@@ -187,9 +187,9 @@ def cmd_notes_update(args: argparse.Namespace) -> int:
|
||||
svc = _service()
|
||||
def _op():
|
||||
original = svc.get_note(args.ticket_id, args.note_id)
|
||||
result = svc.update_note(args.ticket_id, args.note_id, message=args.message, type_id=args.type_id, internal=args.internal, status_id=args.status_id, note_date=args.note_date)
|
||||
result = svc.update_note(args.ticket_id, args.note_id, message=args.message, type_id=args.type_id, internal=not args.external, status_id=args.status_id, note_date=args.note_date)
|
||||
return result, {"original_note": original, "ticket_id": args.ticket_id, "note_id": args.note_id}
|
||||
_print(_run_write("notes.update", {"ticket_id": args.ticket_id, "note_id": args.note_id, "message": args.message, "type_id": args.type_id, "internal": args.internal, "status_id": args.status_id, "note_date": args.note_date}, _op))
|
||||
_print(_run_write("notes.update", {"ticket_id": args.ticket_id, "note_id": args.note_id, "message": args.message, "type_id": args.type_id, "internal": not args.external, "status_id": args.status_id, "note_date": args.note_date}, _op))
|
||||
return 0
|
||||
|
||||
|
||||
@@ -293,7 +293,7 @@ def build_parser() -> argparse.ArgumentParser:
|
||||
na.add_argument("--message", required=True)
|
||||
na.add_argument("--type-id", type=int, default=1)
|
||||
na.add_argument("--status-id", type=int)
|
||||
na.add_argument("--internal", action="store_true")
|
||||
na.add_argument("--external", action="store_true")
|
||||
na.add_argument("--note-date")
|
||||
na.set_defaults(func=cmd_notes_add)
|
||||
nu = nsub.add_parser("update")
|
||||
@@ -302,7 +302,7 @@ def build_parser() -> argparse.ArgumentParser:
|
||||
nu.add_argument("--message", required=True)
|
||||
nu.add_argument("--type-id", type=int, default=1)
|
||||
nu.add_argument("--status-id", type=int)
|
||||
nu.add_argument("--internal", action="store_true")
|
||||
nu.add_argument("--external", action="store_true")
|
||||
nu.add_argument("--note-date")
|
||||
nu.set_defaults(func=cmd_notes_update)
|
||||
nd = nsub.add_parser("delete")
|
||||
|
||||
Reference in New Issue
Block a user