@JackNavaRow con lo que puedas realmente. Hay muchas opciones para ayudar.... También hay muchas palabras para traducir, o con cualquier cosa que se te ocurra

@JackNavaRow ¿Probaste de sugerir alguna traducción? ¿Qué te pareció?

@g3rv4 0 pendientes ... @g3rv4 delegá!

Qué bárbaro, qué terrible este chico

@Mariano pasó que cuando me desperté habían un montón... entonces, para que no se acumularan revisé... pero después se calmó

lo amigos del turno mañana están traduciendo más

@g3rv4 los amigos del turno mañana están traduciendo todo

@JackNavaRow si te interesa colaborar traduciendo, tendrías que tener todo lo necesario :) y si tenés ganas de escribir código, este puede ser un buen issue para arrancar

@g3rv4 las que viste a la mañana eran muchas de las que aprobé a la noche + lo que empezaron a traducir del pendiente

ahí volvió gracias por tu comentario que fedorqui y Pikoh mencionaron ayer

@Mariano sip sip... y hoy volvieron los países! así que algunas sugerencias de antes aparecieron también

Vi... Ayer SO había perdido como unas 600

estábamos por abajo de 1000... y ahora volvimos :(

pero bueno... esos vaivenes no los podemos evitar... SO va a seguir borrando y agregando las mismas cadenas over and over

@g3rv4 El issue de Traducir mantenga las eliminadas por 5 días, lo dejamos para más adelante, no? :-P

@g3rv4 coding convention! vos queres que tu funcion siga devolviendo un task<int?> o puedo cambiar el tipo de devolucion a un <bool,int?> de forma tal de saber si fallo o no.. pq con el task int, vamos fritos no sea que mi enum se cruce con un id real!

@Mariano es faci! creá el issue

@gbianchi que función?

@g3rv4 ok... Lo mando y vos lo etiquetas good first issue?

dale, vos no podés etiquetar, no?

@g3rv4 Task<int?> CreateSuggestionAsync(int stringId, string suggestion, int userId, UserType userType, bool approve);

@g3rv4 nope

@gbianchi que devuelva un Task<SuggestionCreationResult> donde SuggestionCreationResult sea el enum del que hablábamos

el único que llama la función se termina fijando si suggestionId.HasValue

@g3rv4 ok.. mas el id.. pq si lo crea correcto estas devolviendo el id...

si si.. estaba mirando eso..

naaah, yo no devolvería el id

@g3rv4 a ver que haces con ese id en el front... ya me fijo.. sigan lo suyo..

@gbianchi el backend no lo devuelve... podés mirar StringsController

así que el frontend no lo puede usar :)

apa.. tenes razon.. devuelve un emptyresult.. lesto!!!!

aaah pará pará pará

mala mía

el que debería hacer las validaciones no es el servicio

debería ser el controller

todas estas cosas las podría hacer el controller... me gusta mucho más (porque así el servicio no se tiene que preocupar por el enum, que medio que no le corresponde) @gbianchi

@g3rv4 ok.. yo habia puesto el enum junto con los de la DB y me sonaba raro pero bue.. lo subo.. ya se lo que queres.. ahora me fijo.. igual llegado el caso, hago un PR y lo revisamos desde ahi... pq si tengo que tocar codigo, seguro le voy a pifiar a tus convenciones... perooooooo.. va a funcionar :p

@g3rv4 pero ojo.. que tengo que comprobar cosas contra la DB (como que el ID exista).. y chequearlo contra eso.. te parece bien que lo haga el controler?

Si esta bien asi, lo que hago es despues agregarle un parametro a esto, no voy a ir a bucar ese string de vuelta...

@gbianchi si, porque podés llamar al servicio .GetStringById()

eso no pega en la db, sino en el cache de memoria

@g3rv4 ah perfecto...

pero si, estoy de acuerdo... lo podemos conversar en el PR

los agregué como collaborators para que puedan manejar issues, labels etc

la idea es que todos los PRs vengan de forks igual

es sólo para temas administrativos ;)

Joya

@g3rv4 perfecto

el issue a nivel de front o back?

@JackNavaRow ?

@JackNavaRow cual issue?

@g3rv4 <--

@JackNavaRow ese tiene 2 partes... backend y frontend

la API no soporta DELETE, la ux tampoco :)

lo único que si hay es un enum para guardar la historia

@JackNavaRow también, si te va mucho más la parte de diseño del front, estoy seguro que podés identificar cualquier cosa en la que te interese ayudar y lo hablamos acá en el chat

ah claro! una cosa que no me gusta es como estoy seteando el "current string"... me parece que sería mucho más prolijo depender sólo de la location... no me gusta como está (y tampoco me gustaría hacer una llamada ajax cada vez que alguien abre una sugerencia si el navegador ya tiene el string)

pero... esa es un área del frontend que se puede mejorar seguramente mucho

@g3rv4 llego a casa.. vuelvo a revisar todo.. y te hago el pr del 18.. salen mensajitos claros.. :O

otra cosa que se pueden mejorar: los alerts espantosos!

y tengo panic mode on en cuanto a las code convention.. pero bue.. fue casi mas copy paste que otra cosa...

@g3rv4 NO SABES COMO TE AMARIA jajajaja

@gbianchi \🦄/

no se pueden mostrar modales lindos?

@gbianchi para las sugerencias? si, pero lo tuve que hacer bien ancho para que anduviera bien en el cel

o algun mensaje de esos que salen estan 3 segundos y se esconden?

o decís para los alerts?

el tema con el mensaje es que si lo ponés arriba

@g3rv4 para los alert...

y estás mirando abajo, te perdés el mensaje

debería flotar

otro modal mas?

se puede... PRs para cosas que no tienen issues son bienvenidas

modal arriba de modal arriba de modal

eso es UX

@g3rv4 en react me supera por mucho jajajaja

a duras penas toco esto.. ni se como mira...

ja, lo mismo digo todos los días en mi trabajo

ya descargue el repo voy a verlo!

\o/

Qué grande, pensando en todo

El apellido necesita un 🦄

Y @Edu ya tiene otro lugar donde meter el favicon :-)

esto fue lo más cerca que llegué

Jajaja

> (don't mistake me for my estranged cousin... traducir.fail)

@g3rv4 ah, sí, a mí me hace lo mismo, y es que lo estoy viendo en el celu en español también... Después no aparece en ningún lado

Tx tiene sus problemitas también, no nos olvidemos

Te lo van a empezar a ver cómo usuario en las otras comunidades??

Siempre para adelante!!!

:D sisi, pero está quietito por ahí

@Mariano Si de hecho deja lo completo como me lo habían mencionado

@g3rv4 llegue a casa.. ahora chequeo todo.. y hago el PR! weeeee

@gbianchi yo terminé el día y me voy a correr

Siento que se mira algo extraño a como me lo habian mencionado xD

@g3rv4 lo bien que haces

@EduardoJavierMaldonado este me gusta mas...

Hola. Estoy haciendo una prueba de registro como usuario nuevo por medio de correo electrónico.

Hay un botón que dice "Confirmar registración"

@Rubén estas hablando de traducir?

o transifex?

@Rubén ah y hola!! que tal?? tanto tiempo.. :p

@Rubén ah ok..

@Rubén Deberia ser Completar el registro... no?

dificil de buscar.. a ver...

@gbianchi Propuse "Completar registro"

@Rubén arriba lo que esta entre comillas.. dice Completar el registro.. y si lo dejamos igual?? igual esa cadena es mas parecida..

@gbianchi Sería más eficiente así, de otra forma habría que hacer dos cambios. En cualquier caso ¿no debería ser una única cadena para ambos?

@Rubén le estas pidiendo mucho a SO.. Mariano y Gerv pueden contarte de los problemas.. las cadenas no las decidimos nosotros.. cada cadena viene por separado :(

@gbianchi No me digas más. ¿Sabes si se pueden editar las sugerencias?

@Rubén todavia no, pero podes hacer tu propia sugerencia...

y se acepta la "mejor"

@gbianchi Enviada :)

@Rubén si queres pasame los id que vas tocando.. yo tengo permisos para aprobar :p

se ve que me tienen confianza.. lo mal que hacen

jajajaja

@gbianchi ¡Por Dios! ¡A quien se le ocurre tal barbaridad!

:P

@Rubén para yo apruebo.. pero el final es de los mods...

@gbianchi e3e37f3027a27b1ea727d6e23f4f27ac

@Rubén lesto...

necesitamos que un mod lo pase

ahi te aprobe la otra...

gracias por aportar!! :p

@gbianchi :)

@gbianchi Otra cosas que encontré, el asunto del correo de confirmación dice

Completar registro en Stack Overflow en español

No aparece como cadena.

Reportado como bug

@Rubén ehh.. habia un tag especifico... cual era...