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.

Sign up for GitHub

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

Jump to bottom

EH-1518: Käsittele 404-virhetilanne Service Ticketiä haettaessa #560

Merged
merged 1 commit into from
Nov 22, 2023
Merged

EH-1518: Käsittele 404-virhetilanne Service Ticketiä haettaessa #560

merged 1 commit into from
Nov 22, 2023

Conversation

tomikat
Copy link
Contributor

@tomikat tomikat commented Nov 15, 2023 •
edited by severij
Loading

Kuvaus muutoksista

Käsitellään tilanne/tilanteet, jossa CAS palauttaa 404 haettaessa uutta Service Ticketiä. Näitä tapahtunee useimmiten sen jälkeen, kun CAS-palvelu on käynnistetty uudelleen.

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

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ä
  • Yli jääneet kehityskohteet on tiketöity

@tomikat tomikat marked this pull request as ready for review November 15, 2023 13:05
@tomikat tomikat merged commit 7bfe6d1 into master Nov 22, 2023
1 check passed
@tomikat tomikat deleted the EH-1518 branch November 22, 2023 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@severij severij severij approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tomikat @severij