ferlagod/rocinante_android

Watch

Fork 2

Empezar un libro dejaba la lectura sin fecha #26

Closed

HBB wants to merge 0 commits from HBB:pr/fecha-inicio into main

HBB commented

2026-08-14 21:46:16 +00:00

First-time contributor

El formulario de la web manda la fecha del día al cambiar el estado de lectura.
La app no mandaba ninguna, así que BookWyrm daba de alta la lectura sin
fechas y luego no había de dónde sacar ni cuándo se empezó ni cuándo se
terminó: había que escribirlas a mano, libro por libro.

Se manda la que corresponda al paso que se da: start_date al empezar,
finish_date al terminar, stopped_date al dejarlo.

Para probarlo: marcar un libro como «empezar a leer» desde la app y mirar sus
fechas de lectura en la web.

Dos archivos y suelto del resto. Las otras ramas que mando van encima de esta,
pero esta va sola sobre main.

El formulario de la web manda la fecha del día al cambiar el estado de lectura. La app no mandaba ninguna, así que BookWyrm daba de alta la lectura **sin fechas** y luego no había de dónde sacar ni cuándo se empezó ni cuándo se terminó: había que escribirlas a mano, libro por libro. Se manda la que corresponda al paso que se da: `start_date` al empezar, `finish_date` al terminar, `stopped_date` al dejarlo. Para probarlo: marcar un libro como «empezar a leer» desde la app y mirar sus fechas de lectura en la web. Dos archivos y suelto del resto. Las otras ramas que mando van encima de esta, pero esta va sola sobre main.

HBB added 1 commit

2026-08-14 21:46:16 +00:00

fix: empezar un libro sin fecha de inicio cf42d493e7

El formulario de la web manda la fecha del día al cambiar el estado de lectura;
la app no mandaba ninguna, así que la lectura quedaba dada de alta **sin fechas**
y había que escribirlas después a mano, libro por libro. Se manda la que
corresponda al paso que se da: inicio, fin o abandono.

HBB commented

2026-08-14 22:01:37 +00:00

Author

First-time contributor

Mando cinco ramas de una vez, y van una encima de otra porque tocan los mismos
archivos; sueltas chocarían entre ellas. El orden es:

#26 — la fecha de inicio al empezar un libro
#27 — la ficha del libro: copiar datos, pestañas, enlaces y autores
#28 — el perfil: las cifras llevan a los libros, y lo que falta se puede poner
#29 — estanterías propias y favoritos
#30 — cuánto falta para terminar, al ritmo que se lleva

Cada una se puede leer y rechazar por su cuenta. Fusionarlas fuera de ese orden
pide resolver conflictos a mano.

Mientras las de más arriba sigan abiertas, las de más abajo enseñan también sus
commits: cada una dice en su texto cuáles son los suyos. Eso se arregla solo
según se vayan fusionando.

Si alguna no interesa, dímelo y rehago las de después sin ella.

Mando cinco ramas de una vez, y van una encima de otra porque tocan los mismos archivos; sueltas chocarían entre ellas. El orden es: 1. #26 — la fecha de inicio al empezar un libro 2. #27 — la ficha del libro: copiar datos, pestañas, enlaces y autores 3. #28 — el perfil: las cifras llevan a los libros, y lo que falta se puede poner 4. #29 — estanterías propias y favoritos 5. #30 — cuánto falta para terminar, al ritmo que se lleva Cada una se puede leer y rechazar por su cuenta. Fusionarlas fuera de ese orden pide resolver conflictos a mano. Mientras las de más arriba sigan abiertas, las de más abajo enseñan también sus commits: cada una dice en su texto cuáles son los suyos. Eso se arregla solo según se vayan fusionando. Si alguna no interesa, dímelo y rehago las de después sin ella.

HBB commented

2026-08-14 22:18:35 +00:00

Author

First-time contributor

Con estas cinco se me acaban las ideas: no se me ocurre nada más que quiera
cambiar de la aplicación.

Lo único que quedaba en mi lista era crear un libro desde la app, y ahí ya
dejaste escrito el porqué en 15c2e38: sin API de creación, emular el formulario
de Django es frágil y se rompe con cualquier actualización del servidor. El
argumento es bueno y no lo discuto. A mí sacar a la gente al navegador a mitad
de camino me sigue resultando torpe, pero eso es un gusto mío, y el que va a
mantener esto eres tú.

De aquí en adelante mandaré poca cosa, y casi siempre correcciones de algo con
lo que me tropiece usándola. Ha sido un gusto.

Con estas cinco se me acaban las ideas: no se me ocurre nada más que quiera cambiar de la aplicación. Lo único que quedaba en mi lista era crear un libro desde la app, y ahí ya dejaste escrito el porqué en 15c2e38: sin API de creación, emular el formulario de Django es frágil y se rompe con cualquier actualización del servidor. El argumento es bueno y no lo discuto. A mí sacar a la gente al navegador a mitad de camino me sigue resultando torpe, pero eso es un gusto mío, y el que va a mantener esto eres tú. De aquí en adelante mandaré poca cosa, y casi siempre correcciones de algo con lo que me tropiece usándola. Ha sido un gusto.

ferlagod commented

2026-08-15 14:49:54 +00:00

Owner

El placer ha sido mio, me has ayudado una barbaridad.
Es cierto que sacar al usuario para crear el libro no es lo mejor, pero quiero ver bien como hacerlo sin que ocasione errores...

Muchas gracias, ya estan fusionadas

El placer ha sido mio, me has ayudado una barbaridad. Es cierto que sacar al usuario para crear el libro no es lo mejor, pero quiero ver bien como hacerlo sin que ocasione errores... Muchas gracias, ya estan fusionadas

ferlagod closed this pull request

2026-08-15 14:49:54 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.

No reviewers

No labels

No milestone

No project

No assignees

2 participants

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

ferlagod/rocinante_android!26

No description provided.

Rows
Columns