verdnatura/salix
verdnatura
/
salix
3
4
Fork 0
Code Issues Pull Requests 58 Releases Wiki Activity

5934-clientSms #1675

Merged
alexm merged 5 commits from 5934-clientSms into dev 2023-07-20 07:01:02 +00:00
Conversation 0 Commits 5 Files Changed 9 +5 -8
3 changed files with 5 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit ab95ef74a9 - Show all commits

6
db/changes/233001/00-clientSms.sql
View File

@ -7,9 +7,9 @@ CREATE TABLE `vn`.`clientSms` (
KEY `clientSms_FK_1` (`smsFk`), KEY `clientSms_FK_1` (`smsFk`),
CONSTRAINT `clientSms_FK` FOREIGN KEY (`clientFk`) REFERENCES `client` (`id`) ON UPDATE CASCADE, CONSTRAINT `clientSms_FK` FOREIGN KEY (`clientFk`) REFERENCES `client` (`id`) ON UPDATE CASCADE,
CONSTRAINT `clientSms_FK_1` FOREIGN KEY (`smsFk`) REFERENCES `sms` (`id`) ON UPDATE CASCADE CONSTRAINT `clientSms_FK_1` FOREIGN KEY (`smsFk`) REFERENCES `sms` (`id`) ON UPDATE CASCADE
) ENGINE=InnoDB AUTO_INCREMENT=2 DEFAULT CHARSET=utf8mb3 COLLATE=utf8mb3_unicode_ci; ) ENGINE=InnoDB AUTO_INCREMENT=1 DEFAULT CHARSET=utf8mb3 COLLATE=utf8mb3_unicode_ci;
vicent marked this conversation as resolved Outdated
alexm commented 2023-07-19 05:08:42 +00:00
Outdated
Review
Copy Link

AUTO_INCREMENT=2 esta be?

AUTO_INCREMENT=2 esta be?
INSERT INTO `salix`.`ACL` (`model`, `property`, `accessType`, `permission`, `principalType`, `principalId`) INSERT INTO `salix`.`ACL` (`model`, `property`, `accessType`, `permission`, `principalType`, `principalId`)
VALUES VALUES
('ClientSms', '*', 'READ', 'ALLOW', 'ROLE', 'employee'), ('ClientSms', 'find', 'READ', 'ALLOW', 'ROLE', 'employee'),
vicent marked this conversation as resolved Outdated
alexm commented 2023-07-19 05:09:20 +00:00
Outdated
Review
Copy Link

De moment donali els especifics https://loopback.io/doc/en/lb2/Controlling-data-access.html, encomter de *

De moment donali els especifics https://loopback.io/doc/en/lb2/Controlling-data-access.html, encomter de *
('ClientSms', '*', 'WRITE', 'ALLOW', 'ROLE', 'employee'); ('ClientSms', 'create', 'WRITE', 'ALLOW', 'ROLE', 'employee');

3
modules/client/back/models/client-sms.json
View File

@ -14,9 +14,6 @@
}, },
"clientFk": { "clientFk": {
"type": "number" "type": "number"
},
"smsFk": {
"type": "number"
} }
}, },
vicent marked this conversation as resolved
alexm commented 2023-07-19 05:11:16 +00:00
Review
Copy Link

Me sona que quan ja tens la relacio baix, no cal possar tambe el camp en properties

Me sona que quan ja tens la relacio baix, no cal possar tambe el camp en properties
"relations": { "relations": {

4
modules/client/front/sms/index.html
View File

@ -3,7 +3,7 @@
url="ClientSms" url="ClientSms"
link="{clientFk: $ctrl.$params.id}" link="{clientFk: $ctrl.$params.id}"
filter="::$ctrl.filter" filter="::$ctrl.filter"
data="clientSmss" data="clientSmsList"
limit="20" limit="20"
auto-load="true"> auto-load="true">
</vn-crud-model> </vn-crud-model>
@ -20,7 +20,7 @@
</vn-tr> </vn-tr>
</vn-thead> </vn-thead>
<vn-tbody> <vn-tbody>
<vn-tr ng-repeat="clientSms in clientSmss"> <vn-tr ng-repeat="clientSms in clientSmsList">
vicent marked this conversation as resolved Outdated
alexm commented 2023-07-19 05:19:34 +00:00
Outdated
Review
Copy Link

Si ClientSms representa varios mensajes SMS de un cliente, un nombre más apropiado para la variable sería clientSmsList, clientSmsArray, o simplemente clientSms. Esto indicaría que la variable contiene una lista o un conjunto de mensajes SMS de un cliente.

El uso de List o Array al final del nombre de la variable es una convención común en programación para indicar que la variable es una colección de elementos.

Por otro lado, si prefieres no indicar el tipo de dato en el nombre de la variable, simplemente usar clientSms (en plural) también sería adecuado, ya que en inglés, la forma plural de "SMS" es también "SMS".

Jo gastaria clientSmsList

Si ClientSms representa varios mensajes SMS de un cliente, un nombre más apropiado para la variable sería clientSmsList, clientSmsArray, o simplemente clientSms. Esto indicaría que la variable contiene una lista o un conjunto de mensajes SMS de un cliente. El uso de List o Array al final del nombre de la variable es una convención común en programación para indicar que la variable es una colección de elementos. Por otro lado, si prefieres no indicar el tipo de dato en el nombre de la variable, simplemente usar clientSms (en plural) también sería adecuado, ya que en inglés, la forma plural de "SMS" es también "SMS". Jo gastaria clientSmsList
<vn-td> <vn-td>
<span class="link" ng-click="workerDescriptor.show($event, clientSms.sms.senderFk)"> <span class="link" ng-click="workerDescriptor.show($event, clientSms.sms.senderFk)">
{{::clientSms.sms.sender.name}} {{::clientSms.sms.sender.name}}