Skip to content
Snippets Groups Projects

add Certificate manager

Merged Vladimir Golovchenko requested to merge CertificateManager into develop
2 unresolved threads

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
103 130 target: dapServiceController
104 131 onNetworksListReceived:
105 132 {
133 if (!networkList)
134 console.error("networkList is empty")
135
106 136 for(var n=0; n < Object.keys(networkList).length; ++n)
107 137 {
108 138 dapServiceController.CurrentNetwork = networkList[0];
109 dapServiceController.IndexCurrentNetwork = 0;
139 dapServiceController.IndexCurrentNetwork = 0; //тут разве не должно быть n? или этой строки вообще недолжно быть тут. А если список сетей пуст?
  • Опять комментарии на русском. Кроме того, у нас понятие current network хотелось бы избежать вообще, так как мы работаем сразу со всеми сетями одновременно

  • Please register or sign in to reply
  • 52 }
    53 }
    54
    55
    56 ListModel {
    57 id: certificateInfo
    58 //format key, keyView, value
    59 //keys: [certName, certSignatureType] - required
    60 // [creation_date, domain, expiration_date, organization, fullname, email, description] - optional
    61
    62 }
    63
    64
    65 ListModel {
    66 id: createCertificateOptional
    67 //вообще эти ключи нужно вынести в общее перечисление
  • mentioned in commit 5cd32a68

  • Please register or sign in to reply
    Loading