Skip to content
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

Merged
merged 4 commits into from
Jan 13, 2025
Merged

EH-1757: Fix resend functionality #664

merged 4 commits into from
Jan 13, 2025

Conversation

pkalliok
Copy link
Contributor

@pkalliok pkalliok commented Jan 9, 2025

Kuvaus muutoksista

After these changes, the resend-aloitusherate and resend-paattoherate APIs (there are two of each) do the following things for the HOKSen created between the given dates:

  • update palaute status and info in DB, for instance if opiskeluoikeus info has changed in between
  • resend the SQS message to heratepalvelu.

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

  • Build onnistuu ilman virheitä
  • Toiminnallisuuden kattavat yksikkötestit on tehty osana PR:ia
  • PR:n sisältämät muutokset noudattavat sovittuja koodikäytänteitä
  • Koodi on riittävästi dokumentoitu tai se on muuten yksiselitteistä
  • Nimet (muuttujat, funktiot, ...) kuvaavat koodia hyvin

Katselmoijat tarkastavat, että yllä mainitut kohdat toteutuvat

Ennen mergeämistä master-haaralle

  • Vähintään yksi kehittäjä on katselmoinut ja hyväksynyt muutokset
    • Jos muutoksilla voi jotain rikkoessaan olla kauaskantoiset vaikutukset, kannattaa muutokset hyväksyttää useammalla katselmoijalla
  • Katselmoijien esittämät muutosehdotukset on huomioitu
  • Muutokset on testattu QA-ympäristössä
    • Testausohje kirjoitettu
    • Testaus delegoitu OPH:lle mikäli mahdollista
  • Yli jääneet kehityskohteet on tiketöity

Copy link
Contributor

@severij severij left a 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.

src/oph/ehoks/palaute/opiskelija.clj Outdated Show resolved Hide resolved
Copy link
Contributor

@severij severij left a 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 👍

@pkalliok
Copy link
Contributor Author

Laitan testiin.

@pkalliok
Copy link
Contributor Author

Testaus: katsoin uusimman ajon ajankohdan, se on käynnistetty yhdeksältä tänään (maanantaina):

2025-01-13T09:08:37.400+02  INFO [qtp959133997-1605] oph.ehoks.heratepalvelu:0 - Sending 9 (limit 2000) hoksit between 2024-07-01 and 2025-01-13

Kaikki kyseisen ajon HOKSit todettiin jo lähetetyiksi:

2025-01-13T09:08:38.050+02  INFO [qtp959133997-1605] oph.ehoks.palaute.opiskelija:0 - Initial state for :aloituskysely for HOKS 45071 will be :ei-luoda-ollenkaan because of :jo-lahetetty in :id

Tietokannassa ei ole duplikaatteja (tai ylipäänsä uusia herätteitä) asennuksen jälkeen:

ehoks=> select * from palautteet where created_at > '2025-01-10 15:00:00';
id,created_at,updated_at,deleted_at,herate_source,heratepvm,kyselytyyppi,tila,kyselylinkki,arvo_tunniste,voimassa_alkupvm,voimassa_loppupvm,jakson_yksiloiva_tunniste,hoks_id,nippu_id,koulutustoimija,hankintakoulutuksen_toteuttaja,toimipiste_oid,suorituskieli,tutkintotunnus,tutkintonimike

Sen sijaan siltä ajalta, joka oli siivous-PR:n ja korjaus-PR:n asennuksen välissä, duplikaatteja on tullut entiseen tapaan:

ehoks=> select date(created_at), count(*) as count from palautteet group by date(created_at) order by count desc;
date,count
2025-01-10,140
2025-01-09,108
2024-09-11,29
2024-09-10,17
2024-10-23,17
2024-09-09,12
[...]

@pkalliok
Copy link
Contributor Author

Mergeän tämän.

@pkalliok pkalliok merged commit d9238a5 into master Jan 13, 2025
2 checks passed
@pkalliok pkalliok deleted the pkalliok-EH-1757 branch January 13, 2025 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants