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

feat: HASSU-741 nahtavillaolo ilmon vastaanottajat #265

Merged

Conversation

ValheKouneli
Copy link
Contributor

No description provided.

@ValheKouneli ValheKouneli force-pushed the feature/HASSU-741-nahtavillaolo-ilmon-vastaanottajat branch from 7e4cb83 to 1719c15 Compare June 13, 2022 12:32
Comment on lines 19 to 26
kuulutusPaiva: projekti?.nahtavillaoloVaihe?.kuulutusPaiva,
kuulutusVaihePaattyyPaiva: projekti?.nahtavillaoloVaihe?.kuulutusVaihePaattyyPaiva,
muistutusoikeusPaattyyPaiva: projekti?.nahtavillaoloVaihe?.muistutusoikeusPaattyyPaiva,
ilmoituksenVastaanottajat: defaultVastaanottajat(
projekti,
projekti.nahtavillaoloVaihe?.ilmoituksenVastaanottajat,
kirjaamoOsoitteet
),
Copy link
Contributor

Choose a reason for hiding this comment

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

Täältä puuttuu nyt niitä resetoitavia kenttiä. Hankkeen kuvaus, yhteystiedot

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ääh, joku mergekonflikti!

Copy link
Contributor

@tkork tkork left a comment

Choose a reason for hiding this comment

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

defaultValues funkkariin olisi hyvä lisätä puuttuvat kentät, jotta ne täyttyy. Hankkeenkuvaukset ja yhteystiedot ei ainakaan ollut siellä

@ValheKouneli ValheKouneli merged commit 43126bb into main Jun 13, 2022
@ValheKouneli ValheKouneli deleted the feature/HASSU-741-nahtavillaolo-ilmon-vastaanottajat branch June 13, 2022 13:03
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