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

Tuloselvitysten liitteet yksitellen #6319

Merged
merged 17 commits into from
Feb 12, 2025
Merged

Tuloselvitysten liitteet yksitellen #6319

merged 17 commits into from
Feb 12, 2025

Conversation

akheron
Copy link
Contributor

@akheron akheron commented Jan 31, 2025

Kuntalaisen puolella uusille tuloselvityksille:

  • Näytetään upload-komponentti jokaiselle liitetyypille erikseen siinä kohdassa sivua, jossa liitettä pyydetään
  • Näytetään sivun alareunassa, mitkä liitteet puuttuvat, sekä linkit sivulla oikeaan kohtaan siirtymiseen
  • Ei sallita tuloselvityksen lähettämistä ennen kuin kaikille vaadituille liitetyypeille on ladattu vähintään yksi liite

Kuntalaisen puolella vanhoille tuloselvityksille, joissa on vähintään yksi liite:

  • Näytetään vanhaan malliin yksi upload-komponentti sivun alareunassa

Teknisesti tämä on toteutettu asettamalla uusien tuloselvityksen liitteille tyyppi. Upload-komponentteja on yksi per liitetyyppi. "Vanhat tuloselvitykset" eli tuloselvitykset, joissa on vähintään yksi liite ilman tyyppiä, pysyvät ~ennallaan.
Tuen vanhoille tuloselvityksille muokkauspuolella voi poistaa sitten, kun ei ole enää käsittelemättömiä tuloselvityksiä, joissa on tyypittömiä liitteitä.

Tuloselvityksen muokkaaminen lähettämisen jälkeen on siirretty tuloselvityksen katselusivulle. Sen sijaan että lomakkeessa olisi lähes kaikki kentät harmaana, näytetään kastelusivulla lisätietokenttä sekä liitteet muokattavana, ja kaikki muut tiedot vain katseltavana kuten ennenkin.

@akheron akheron changed the title Tulotietojen liitteet yksitellen Tuloselvitysten liitteet yksitellen Jan 31, 2025
@akheron akheron force-pushed the attachments-one-by-one branch 3 times, most recently from d2c1eb3 to f676b0b Compare February 3, 2025 12:33
@akheron akheron marked this pull request as ready for review February 3, 2025 12:33
@akheron akheron added the enhancement Uusi toiminnallisuus tai parannus label Feb 3, 2025
@Joosakur
Copy link
Contributor

Joosakur commented Feb 5, 2025

Bugi:

  • tee tuloselvitys, lisää liitteitä ja lisää myös "muu liite", lähetä
  • siirry antamaan lisätietoja
  • muut aiemmin ladatut liitteet näkyy, mutta muu liite ei (virkailijan puolella sekin näkyy)

@akheron akheron force-pushed the attachments-one-by-one branch from 6c9d40a to 13528f1 Compare February 12, 2025 06:58
@akheron
Copy link
Contributor Author

akheron commented Feb 12, 2025

Bugi korjattu 👍

@akheron akheron merged commit a4bd7e6 into master Feb 12, 2025
28 of 29 checks passed
@akheron akheron deleted the attachments-one-by-one branch February 12, 2025 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Uusi toiminnallisuus tai parannus
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants