Deptos y roles listo para mostrar y crear.

[AleKiller21]

Tanto como los departamentos y roles ya se pueden crear y tambien mostrar al momento de cargar la pagina. Se utilizo Ajax y JQuery.

[danielzelayadev]

Cuando cargan los departamentos al view, validen que no se carguen "descripciones" repetidas. Por ejemplo, puede haber mas de un registro de Motorista pero con distinto id. Entonces no se trata de cargar todos los registros de la tabla sino de cargar cada descripcion unica. Se entiende? Imagino que no es tanto rollo esta validacion

[KelviNosse]

En todo caso seria entonces no crear un departamento con descripciones iguales, ya que no tendria sentido tener dos departamentos que sean Motorista por ejemplo si ya existe uno xD

[danielzelayadev]

Es la DB la que lo hace

[KelviNosse]

Validar que no contenga descripcion repetida, antes de crearla

[danielzelayadev]

Es necesario para relacionar a los usuarios con sus roles y otras cosas, entonces toca hacer esa validacion. Es mi unica observacion ya lo testee y me gusto mucho. Arreglenlo y de ahi tienen mi si

[danielzelayadev]

Basicamente validar que no esten displaying mas de una vez el mismo depto y/o rol

[KelviNosse]

Osea que es necesario que existan dos departamentos iguales?

[danielzelayadev]

Es que asi funcionan las bases de datos. Para relacionar crear registros iguales con keys diferentes

[danielzelayadev]

Por eso en la base de datos vas a ver que hay mas de un registro con Motorista o etc. pero el key nunca va a ser el mismo.

[danielzelayadev]

Uds solo validen que si el description del depto y/o rol ya lo pusieron en el view no lo vuelvan a poner

[AleKiller21]

Estoy trabajando en eso

[AleKiller21]

No! Tiene que ser rolname por unos rollos de js en el html

[AleKiller21]

Dejalo como rolname, tuvimos que cambiarle el nombre porque no se lo podiamos cambiar en el js

[danielzelayadev]

@AleKiller21 ensename donde esta ese rollo del js. Creeme que ese refactor que se hicieron les creo estos conflictos

[AleKiller21]

Kelvin sabe donde esta. Antes de cambiarle el nombre Resharper me dijo donde se utilizaba esa variable y solo era en tres funciones del Web Service. Todo estaba funcionando bien

[danielzelayadev]

Si pero los commits antiguos no usaban eso entonces detecto conflicto. Hay que hacer refactors lo menos posible

[danielzelayadev]

@KelviNosse ensename donde esta eso de js porfa

[AleKiller21]

Perame, y como voy le voy hacer rename en los commits antiguos? El refactor ya se lo habiamos hecho y todo funcionaba.

[danielzelayadev]

No tenes que hacer rename en los commits antiguos. Y, en efecto, causa de los conflictos para este merge es ese refator porque, como te decia, la ultima version de develop no sabe que ahora Descripcion en Roles se llama 'rolname'. Entonces git no sabe que hacer. Un simple refactor resolveria eso. Si tan solo pudiera ver cual es el rollo con js ayudaria para arreglarlo

[AleKiller21]

Ya no va a mostrar duplicados. Por cierto, segun el libro el id y el name debian tener el mismo nombre del atributo en la clase modelo.

[danielzelayadev]

Ya veo, es que en casi todos los sitios que visite hoy solo usaban el name attribute

[AleKiller21]

Ya veo. Es algo que podemos que podemos examinar pero no ahorita.

[danielzelayadev]

Bueno ahora solo queda lo de rename rolname. Yo digo que es lo mejor

[AleKiller21]

Ok. Pero exactamente como le doy rename. Osea, ahorita funciona pero como hago para los otros commits?

[danielzelayadev]

Te confundiste un poquito con lo de los commits. Luego te explico a lo que me referia. Andate a la definicion de la clase -> right click -> refactor -> rename

[AleKiller21]

En RolesMirror. Pero si le cambio el nombre de vuelta como hago con lo de js?

[danielzelayadev]

Cambialo ahi tambien

[danielzelayadev]

Solo es en ese parametro que Kelvin nos enseno

[AleKiller21]

Ok, voy a probar

[AleKiller21]

Lo hice pero ya no me muestra los roles en la lista

[danielzelayadev]

Hacete el push?

[AleKiller21]

Esperen, ya aparecen. Solo estoy viendo otra cosa.

[KelviNosse]

como van? ya modificaron a Descripcion?

[danielzelayadev]

Me alegro @AleKiller21

[danielzelayadev]

Y lol @KelviNosse lee el feed no seas perezoso

[KelviNosse]

ya lei, pero al menos ya estamos a punto de resolver conflict?

[danielzelayadev]

Cuando Ale haga push vamos a ver si se quita el coso de conflictos

[AleKiller21]

Listo

[AleKiller21]

Ya lo cambie devuelta a Descripcion, agrega y muestra

[KelviNosse]

genial, probalo puess lolololol

[danielzelayadev]

Cheque

[KelviNosse]

:+1: :new_moon_with_face:

[AleKiller21]

Ya le di push

[AleKiller21]

Lo probe y funcionaba. No muestra duplicados y el atributo se llama de vuelta Description

[danielzelayadev]

Excelente

[KelviNosse]

Genial, vamos a ver si se resuelve esto de merge

[danielzelayadev]

Bueno eso es todo, apruebo este merge :+1: voy a resolver los conflictos

[KelviNosse]

yo queriaaaa okno, lol dale pues, lets go :dog:

[danielzelayadev]

No padawan, no @KelviNosse

[KelviNosse]

git push -f okno xDDDD