-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
EH-1757: Fix resend functionality #664
Conversation
Also, because the :resend? flag is now effectively a no-op, remove the flag too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tää ei musta nykyisellään toimi oikein, kts. kommentit.
d2afcbc
to
847c006
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Jeps, tää on mun puolesta ok 👍
Laitan testiin. |
Testaus: katsoin uusimman ajon ajankohdan, se on käynnistetty yhdeksältä tänään (maanantaina):
Kaikki kyseisen ajon HOKSit todettiin jo lähetetyiksi:
Tietokannassa ei ole duplikaatteja (tai ylipäänsä uusia herätteitä) asennuksen jälkeen:
Sen sijaan siltä ajalta, joka oli siivous-PR:n ja korjaus-PR:n asennuksen välissä, duplikaatteja on tullut entiseen tapaan:
|
Mergeän tämän. |
Kuvaus muutoksista
After these changes, the
resend-aloitusherate
andresend-paattoherate
APIs (there are two of each) do the following things for the HOKSen created between the given dates:As a sidenote, using the creation date is probably not correct; it will skip HOKSen that were just updated in the last two weeks (or whatever the given dates are).
https://jira.eduuni.fi/browse/EH-1757
Muistilista PR:n tekijälle ja katselmoijille
Ennen asettamista katselmointiin
❗ Katselmoijat tarkastavat, että yllä mainitut kohdat toteutuvat
Ennen mergeämistä
master
-haaralle