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

Add ehoks-heratepalvelu AWS dashboard source #558

Merged
merged 1 commit into from
Nov 9, 2023

Conversation

severij
Copy link
Contributor

@severij severij commented Nov 9, 2023

Kuvaus muutoksista

Lisätty ehoks-heratepalvelu AWS Dashboardin sorsa. Nykyisestä dashboardista voi ottaa dumpin leikepöydälle valitsemalla (dashboardin ollessa avattuna) "Actions" -> "View/edit source" -> "Copy source". Vastaavasti, dashboardin saa tarvittaessa palautettua copy-pasteamalla sourcen JSON-muokkauskenttään ja painamalla "Update".

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

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

@severij
Copy link
Contributor Author

severij commented Nov 9, 2023

Tässä lisätään vaan tuon dashboardin JSON resources-hakemistoon. Ei vaadi katselmointia, joten mergeän.

@severij severij merged commit 95592ba into master Nov 9, 2023
1 check passed
@severij severij deleted the severij/aws-dashboard-source branch November 9, 2023 12:42
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.

1 participant