Attention is currently required from: jolly.

dexter has posted comments on this change by jolly. ( 
https://gerrit.osmocom.org/c/onomondo-ipa/+/43035?usp=email )

Change subject: V1.2: Add eimTransactionId to IpaEuiccDataRequest and 
IpaEuiccDataResponse
......................................................................


Patch Set 2: Code-Review+2

(1 comment)

File src/ipa/libipa/proc_euicc_data_req.c:

https://gerrit.osmocom.org/c/onomondo-ipa/+/43035/comment/fdb7a609_de3e7748?usp=email
 :
PS1, Line 216:          IPA_LOGP(SIPA, LINFO, "IPA get EUICC data 
succeeded!\n");
> When looking closer I can see that this code is a bit misleading. […]
As this is not in scope of this patch, we can set this to done. I took a note 
on my todo list so that I can create the ticket later.



--
To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43035?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: onomondo-ipa
Gerrit-Branch: master
Gerrit-Change-Id: I34db78c4eb8461c154459204ec13a2184085e96a
Gerrit-Change-Number: 43035
Gerrit-PatchSet: 2
Gerrit-Owner: jolly <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <[email protected]>
Gerrit-Attention: jolly <[email protected]>
Gerrit-Comment-Date: Thu, 30 Jul 2026 11:11:30 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: dexter <[email protected]>

Reply via email to