humanmade / humanmade/repeatable-posts

Repeating post is marked as such even though a Repeat post is not created for it (due to be scheduled in the past)

Abierto
#35 3 comentarios 0 reacciones 0 asignados Ver en GitHub
enhancement
Lenguaje dominante
PHP
Estrellas
48
Forks
12
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

A Repeat post is created only if date of a Repeating post + it's schedule is in the future.

When you setup a post to be a Repeating post with a schedule such that a Repeat post date happens to be in the past (read example below), the Repeat post is not created. However, the Repeating post is marked as `Repeating`. This is confusing!

**Solution**: an error message should be displayed in such case while trying to save a post with a Repeating schedule. That post won't be saved as `Repeating` post.

Example: main post (that we're wanting to set as a Repeating post) is published on the 10th October and we're setting it to be a `Daily` Repeating post. So a daily repeat post would have been scheduled for the 11th October. However, if present time (when we're saving changes) is anything past the 11th October, say the 12th October - then Repeat post date is in the past and therefore it is not published.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

The issue names no files or tests. Start by tracing the save path for Repeating posts and the logic that calculates the first Repeat post date, then reproduce the daily-schedule example with the repeat date already in the past. Done means saving is rejected with an error and the post is not marked as Repeating.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
php, wordpress
Área
backend, content
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
42/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.