Add in a part_svc.comment, with ability to edit/view

[Jayceh]

orthogonal to the part_pkg.comment.

[Jayceh]

->insert works, but apparently there is currently an issue with ->replace, debugging now

[Jayceh]

Interface usage should now be fixed.

[freeside]

You added the column, and the ability to edit it, but it doesn't do anything else. Can you please share a complete change that shows where this field would be displayed in the interface?

[Jayceh]

I didn't do that yet just waiting to see if it was worth doing based on
your feedback. If you think it fits then yes, I'll finish it and add it to
the pull request so the interface is usable.

On Tuesday, September 8, 2015, Freeside Internet Services, Inc. <
[email protected]> wrote:

You added the column, and the ability to edit it, but it doesn't do
anything else. Can you please share a complete change that shows where this
field would be displayed in the interface?

—
Reply to this email directly or view it on GitHub
#49 (comment).

Jayce^

Preparing Deseret - UtahPreppers
ΜΟΛΩΝ ΛΑΒΕ

[freeside]

Sure, it seems like it fits. I'd have to know specifically where you intend to add it in the interface to provide more feedback than that. I don't think it is ready to merge until it is a complete change that displays in the interface in an appropriate way in additional to the database and edit changes.

[Jayceh]

Agreed on the merge. I posted it just to have some source material to
discuss. Will finish out the rest and submit.

On Tue, Sep 8, 2015 at 6:37 PM, Freeside Internet Services, Inc. <
[email protected]> wrote:

Sure, it seems like it fits. I'd have to know specifically where you
intend to add it in the interface to provide more feedback than that. I
don't think it is ready to merge until it is a complete change that
displays in the interface in an appropriate way in additional to the
database and edit changes.

—
Reply to this email directly or view it on GitHub
#49 (comment).

Jayce^

Preparing Deseret - UtahPreppers
ΜΟΛΩΝ ΛΑΒΕ