jgrulich added a comment.

  In D17834#386573 <https://phabricator.kde.org/D17834#386573>, @enriquem wrote:
  
  > I implemented all of your comments except one. I created a new VPN setting 
wit nm-connection-editor, and it sets it to 0, that is, to 
NetworkManager::Setting::None. Thus, I believe that part is correct.
  
  
  I see. Maybe do not set it at all if one-time password is not used and let NM 
to set the default value? That way we won't be using a wrong value in case this 
changes in future.

REPOSITORY
  R116 Plasma Network Management Applet

REVISION DETAIL
  https://phabricator.kde.org/D17834

To: enriquem, jgrulich
Cc: plasma-devel, kvanton, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

Reply via email to