Skip to content
This repository has been archived by the owner on Feb 7, 2024. It is now read-only.

Feature/ae 1977 #1041

Merged
merged 18 commits into from
Nov 17, 2023
Merged

Feature/ae 1977 #1041

merged 18 commits into from
Nov 17, 2023

Conversation

Juholei
Copy link
Contributor

@Juholei Juholei commented Nov 9, 2023

No description provided.

@Juholei Juholei force-pushed the feature/AE-1977 branch 4 times, most recently from cdc68b0 to 2b6527d Compare November 14, 2023 12:42
@Juholei Juholei marked this pull request as ready for review November 14, 2023 12:43
- Currently just returns 200, no document yet
…päätös document

- Will be added as a separate document later
…li when creating käskypäätös / varsinainen päätös toimenpide
- Hallinto-oikeus attachments are sent to asha when creating käskypäätös / varsinainen päätös toimenpide
- Refactor log-toimenpide! to multiarity function from variadic
Copy link
Contributor

@solita-antti-mottonen solita-antti-mottonen left a comment

Choose a reason for hiding this comment

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

Ashalle useampi dokumentti on vissiin täsmälleen samanlainen keissi kuin aiemminkin on ollut?

:type (-> processing-action :document :type)
:name (-> processing-action :document :filename)}]))

(doseq [attachment attachments]
Copy link
Contributor

Choose a reason for hiding this comment

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

Nyt ei vissiin testata että liitteet liitetään? Ei taideta kyllä testata dokumenttejakaan sen puoleen.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Joo, en lähtenyt perehtymään siihen ashalle lähtevän sanoman testaamiseen. Onko sulla kokemusta miten se onnistuisi kätevästi?

Copy link
Contributor

Choose a reason for hiding this comment

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

Kätevästi ei. Kai se samankaltainen pyyntö on kuin muutkin asha-testin pyynnöt, mutta vaatisi oikeanlaisen jotain vakoilua että saisi oikeanlaisen kysymys ja vastaus formaatin

@Juholei
Copy link
Contributor Author

Juholei commented Nov 17, 2023

Ashalle useampi dokumentti on vissiin täsmälleen samanlainen keissi kuin aiemminkin on ollut?

Ainakin näennäisesti joo. Aiemminkin on mennyt useampi dokumentti per toimenpide, jos osapuolia on useampi. Meidän pään toteutuksen perusteella siinä ei näyttäisi olevan väliä mitä sinne tunkee, mutta pitää toki vielä testata ashan kanssa testiympäristössä.

@Juholei Juholei merged commit f43f106 into develop Nov 17, 2023
4 checks passed
@Juholei Juholei deleted the feature/AE-1977 branch November 17, 2023 13:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants