@miconda commented on this pull request.


> @@ -1290,7 +1290,7 @@ modparam("acc", "cdrs_table", "acc_cdrs")
                                microseconds in time_exten.</para>
                </listitem>
                <listitem>
-                       <para><emphasis>2</emphasis> - save seconds.milliseconds
+                       <para><emphasis>2</emphasis> - save seconds.microseconds

Shouldn't be better a new `time_mode` value (e.g., 5) for this, instead of 
changing the behaviour for 2? Existing deployments might be affected, requiring 
probably external billing apps to be upgraded as well.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4447#pullrequestreview-3411661784
You are receiving this because you are subscribed to this thread.

Message ID: <kamailio/kamailio/pull/4447/review/[email protected]>
_______________________________________________
Kamailio - Development Mailing List -- [email protected]
To unsubscribe send an email to [email protected]
Important: keep the mailing list in the recipients, do not reply only to the 
sender!

Reply via email to