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-1699: Add hoks_id and yksiloiva_tunniste to select_%_tyopaikkajaksot_active_between.sql queries #639

Merged
merged 2 commits into from
Sep 2, 2024

Conversation

severij
Copy link
Contributor

@severij severij commented Aug 29, 2024

Kuvaus muutoksista

EH-1699 muutosten myötä, Herätepalvelussa aletaan käyttämään jaksojen yksilöintiin (hoks_id, yksiloiva_tunniste)-kombinaatiota. Tämän mahdollistamiseksi, sekä hoks_id että yksiloiva_tunniste täytyy sisällyttää mukaan select_%_tyopaikkajaksot_active_between.sql-kyselyihin.

https://jira.eduuni.fi/browse/EH-1699

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

@severij severij requested a review from tomikat August 29, 2024 08:44
Copy link
Contributor

@tomikat tomikat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hyvältä näyttää! 👍

@severij
Copy link
Contributor Author

severij commented Sep 2, 2024

Testauskommentteja: Opetushallitus/heratepalvelu#284 (comment)

@severij severij merged commit 55e59ae into master Sep 2, 2024
2 checks passed
@severij severij deleted the severij/EH-1699 branch September 2, 2024 12:32
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