[devel] [PATCH 0/1] Review Request for ntf: Update README for NTF notification enhancement [#3201]

2020-07-15 Thread Thanh Nguyen
Summary: ntf: Update README for NTF notification enhancement [#3201] Review request for Ticket(s): 3201 Peer Reviewer(s): *** LIST THE TECH REVIEWER(S) / MAINTAINER(S) HERE *** Pull request to: *** LIST THE PERSON WITH PUSH ACCESS HERE *** Affected branch(es): develop Development branch:

[devel] [PATCH 1/1] ntf: Update README for NTF notification enhancement [#3201]

2020-07-15 Thread Thanh Nguyen
The README file is update to reflect the implementation in [#3169] and [#3200] --- src/ntf/ntfimcnd/README | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/ntf/ntfimcnd/README b/src/ntf/ntfimcnd/README index 2809e7eb9..cf0619979 100644 --- a/src/ntf/ntfimcnd/README +++

Re: [devel] [PATCH 1/1] ntf: Handle IMM multiple value attribute in notification [#3200]

2020-07-15 Thread Thanh Nguyen
Thx Minh, When it gets ACKs from others, please help to merge. In the mean time, I will update the MR with pro/cons. This kind of discussion will also go to read me file. Cheers, Thanh -Original Message- From: Minh Hon Chau Sent: Wednesday, 15 July 2020 10:29 PM To: Thanh Nguyen ;

Re: [devel] [PATCH 1/1] ntf: Handle IMM multiple value attribute in notification [#3200]

2020-07-15 Thread minhchau
Hi aThanh, Ack from me. I guess we will document the README with this change to mention the pros/cons of either not to fill any old values or just to fill some of them, so that the reason of this change can be clarified? Thanks Minh On 14/7/20 10:57 am, Thanh Nguyen wrote: When IMM