OY-5034 tyopoydan emailien palastelu #90
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…rajapintaan
Näin vakioihin voi viitata:
- clienttiä käyttävässä koodissa
- rajapinnan javadockeissa
Päivitetty minor-versionumero koska clienttiin tullut uusi "ominaisuus", mutta
yhteensopiva aikaisemman version kanssa.
jkorri
force-pushed
the
OY-5034_tyopoydan_emailien_palastelu
branch
2 times, most recently
from
de6d851 to
2881ff5
Compare
marjakari
reviewed
Jan 14, 2025
kirjasto/README.md
Outdated
| @@ -1,6 +1,8 @@ | |||
| ## Viestinvälityspalvelu kirjasto | |||
|
|
|||
| Kirjaston avulla asiakasjärjestelmät voivat käyttää viestinvälityspalvelua java-rajapinnan läpi. | |||
| Kirjaston avulla asiakasjärjestelmät voivat käyttää viestinvälityspalvelua java-rajapinnan läpi. Käyttöön tarvitaan | |||
| tämä riippuvuus, sekä käyttäjätunnus jolla on tarvittavat oikeudet viestien lähettämiseksi. Kirjaston transitiiviset | |||
There was a problem hiding this comment.
Suggested change
| tämä riippuvuus, sekä käyttäjätunnus jolla on tarvittavat oikeudet viestien lähettämiseksi. Kirjaston transitiiviset | |
| tämä riippuvuus, sekä palvelutunnus jolla on tarvittavat oikeudet viestien lähettämiseksi. Kirjaston transitiiviset |
marjakari
reviewed
Jan 14, 2025
| @@ -53,7 +51,7 @@ object ViestiValidator: | |||
| final val VALIDATION_VASTAANOTTAJA_OSOITE_DUPLICATE = "vastaanottajat: Osoite-kentissä on duplikaatteja: " | |||
| final val VALIDATION_VASTAANOTTAJAN_NIMI_LIIAN_PITKA = "nimi-kenttä voi maksimissaan olla " + VIESTI_NIMI_MAX_PITUUS + " merkkiä pitkä" | |||
| final val VALIDATION_VASTAANOTTAJAN_OSOITE_TYHJA = "sähköpostiosoite-kenttä on pakollinen" | |||
| final val VALIDATION_VASTAANOTTAJAN_OSOITE_INVALID = "sähköpostiosoite ei ole validi sähköpostiosoite" | |||
| final val VALIDATION_VASTAANOTTAJAN_OSOITE_LIIAN_PITKA = "sähköpostiosoite void maksimissaan olla " + " merkkiä pitkä" | |||
There was a problem hiding this comment.
pitikö tähän tulla se max pituus vakiosta?
marjakari
reviewed
Jan 14, 2025
| /** | ||
| * Viestin lähettäjän ja yksittäisten vastaanottajien nimien maksimipituus. | ||
| */ | ||
| static final int VIESTI_NIMI_MAX_PITUUS = 64; |
There was a problem hiding this comment.
Tätä lukua voisi vähän kasvattaa mielenrauhan vuoksi jos ei tuota muita muutoksia? :)
marjakari
reviewed
Jan 14, 2025
| // ei validi sähköpostiosoite ei ole sallittu | ||
| Assertions.assertEquals(Set("Vastaanottaja (nimi: Vallu Vastaanottaja, sähköpostiosoite: Optional[ei validi osoite]): " + ViestiValidator.VALIDATION_VASTAANOTTAJAN_OSOITE_INVALID), | ||
| ViestiValidator.validateVastaanottajat(Optional.of(util.List.of(getVastaanottaja("Vallu Vastaanottaja", "ei validi osoite"))))) | ||
| // liian pitkä sähköpostiosoite ei ole sallittu |
There was a problem hiding this comment.
Tähän voisi tietysti ihan testillä dokumentoida että epävalidi sähköpostiosoite menee validoinneista läpi
marjakari
approved these changes
Jan 14, 2025
jkorri
force-pushed
the
OY-5034_tyopoydan_emailien_palastelu
branch
from
79bdc30 to
19985eb
Compare
- siirretään lähetyksen yhteydessä suoraan virhetilaan
- tehdään näin koska jos validoinnin epäonnistuminen johtaa virheeseen
viestin lähettäminen suurelle joukolle järjestelmästä jossa
osoitteet ei kunnolla validoitu on hankalaa
- Ulkomaalaisilla hakijoilla voi olla hyvinkin pitkiä nimiä
jkorri
force-pushed
the
OY-5034_tyopoydan_emailien_palastelu
branch
from
19985eb to
541921d
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
HUOM! Muiden kuin main-branchin kirjasto deploy poistettava ennen mergeä!!!!