{"action":"created","issue":{"url":"https://api.github.com/repos/ingadhoc/account-payment/issues/29","repository_url":"https://api.github.com/repos/ingadhoc/account-payment","labels_url":"https://api.github.com/repos/ingadhoc/account-payment/issues/29/labels{/name}","comments_url":"https://api.github.com/repos/ingadhoc/account-payment/issues/29/comments","events_url":"https://api.github.com/repos/ingadhoc/account-payment/issues/29/events","html_url":"https://github.com/ingadhoc/account-payment/issues/29","id":194150174,"number":29,"title":"[IMP][9.0] Usar modelo account.payment para implementar account.payment.group","user":{"login":"jjscarafia","id":3016656,"avatar_url":"https://avatars.githubusercontent.com/u/3016656?v=3","gravatar_id":"","url":"https://api.github.com/users/jjscarafia","html_url":"https://github.com/jjscarafia","followers_url":"https://api.github.com/users/jjscarafia/followers","following_url":"https://api.github.com/users/jjscarafia/following{/other_user}","gists_url":"https://api.github.com/users/jjscarafia/gists{/gist_id}","starred_url":"https://api.github.com/users/jjscarafia/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jjscarafia/subscriptions","organizations_url":"https://api.github.com/users/jjscarafia/orgs","repos_url":"https://api.github.com/users/jjscarafia/repos","events_url":"https://api.github.com/users/jjscarafia/events{/privacy}","received_events_url":"https://api.github.com/users/jjscarafia/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-12-07T19:49:21Z","updated_at":"2016-12-10T11:11:32Z","closed_at":null,"body":"La propuesta es que el módulo account_payment_group, en vez de generar un nuevo modelo \"account.payment.group\" implemente toda esa lógica en account.payment agregando un nuevo tipo de pago (payment_type = group). Básicamente quedarían payments con payments hijos\r\n\r\nNotas de desarrollo:\r\n\r\n- Hacer journal_id no required\r\n\r\nVentajas:\r\n\r\n- No haría falta ocultar y generar nuevos menus, si cambiaríamos el dominio de los menus para que se muestren los payments que no tengan payment padre\r\n- La integración con account_document sería más directa, ya que es la misma clase la que tendría el tipo de documento y la numeración\r\n- Si se instala el módulo en una bd con datos, no habría que generar un payment group para cada payment actual ya que se podrían visualizar y consultar sin problemas\r\n- La migración de datos parece que iría más facil entre v8 y v9 (por mismo tema de punto anterior)\r\n- No recuerdo ahora pero había algunas otras que había pensado\r\n\r\nNo muy lindo:\r\n\r\n- Deberíamos hacer que journal_id sea no requerido (no muy odoo way)\r\n- Si se desinstala el módulo habría que proponer borrar todos los payments del tipo group o algo por el estilo"},"comment":{"url":"https://api.github.com/repos/ingadhoc/account-payment/issues/comments/266200675","html_url":"https://github.com/ingadhoc/account-payment/issues/29#issuecomment-266200675","issue_url":"https://api.github.com/repos/ingadhoc/account-payment/issues/29","id":266200675,"user":{"login":"jjscarafia","id":3016656,"avatar_url":"https://avatars.githubusercontent.com/u/3016656?v=3","gravatar_id":"","url":"https://api.github.com/users/jjscarafia","html_url":"https://github.com/jjscarafia","followers_url":"https://api.github.com/users/jjscarafia/followers","following_url":"https://api.github.com/users/jjscarafia/following{/other_user}","gists_url":"https://api.github.com/users/jjscarafia/gists{/gist_id}","starred_url":"https://api.github.com/users/jjscarafia/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jjscarafia/subscriptions","organizations_url":"https://api.github.com/users/jjscarafia/orgs","repos_url":"https://api.github.com/users/jjscarafia/repos","events_url":"https://api.github.com/users/jjscarafia/events{/privacy}","received_events_url":"https://api.github.com/users/jjscarafia/received_events","type":"User","site_admin":false},"created_at":"2016-12-10T11:11:32Z","updated_at":"2016-12-10T11:11:32Z","body":"La idea sería que conceptualmente funcione igual que ahora pero que la clase agrupadora pueda ser la misma clase, otras ventajas que se me ocurren:\r\n\r\n* ahora mismo como está implementado deberíamos crear un menú para las transferencias (ya que son account.payment) que no llevarían un payment.group por arriba, igualmente esto no lo veo tan feo\r\n* Actualmente, los kanban de journals te permiten navegar y llegar a los payments. No podría ser confuso eso?\r\n* Por las dudas deberíamos bloquear la creación de cualquier payment sin payment group salvo para los payments \"transfer\". Porque si se crea alguno va a quedar inaccesible\r\n\r\nEstoy casi seguro que había pensado en otra pero me quedé dormido jeje. Ahora que escribo estas tres en realidad me parece que son mejoras pero son cosas que podemos resolver usando una clase distinta tmb. No me termino de definir...."}}
{"actor":{"display_login":"jjscarafia"}}