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: react frontend #18

Merged
merged 8 commits into from
Sep 11, 2024
Merged

feat: react frontend #18

merged 8 commits into from
Sep 11, 2024

Conversation

saku-koodari
Copy link
Contributor

@saku-koodari saku-koodari commented Sep 10, 2024

Rakennettu React front next.js - teknologialla.

  • Valitsin next.js, koska se on reactin de-facto -suositus. Itse reactin pystytys on helppo tehdä, joten next.js korvaaminen olisi vielä helppoa.
  • Frontti on deployattu nyt static export:ina, koska se ei vaadi nextin omaa bäkkäriä, vaan voidaan käyttää meidän spring boottia.
  • next.js (static exporttina) buildaantuu frontin out/ - kansioon
  • Lisätty pom.xml :ään
  • HUOM! Tässä versiossa ./mvnw spring-boot:run - ei kopioi tiedostoja static:in alle. React -> static kopiointiin olen käyttänyt mvn package - komentoa. Pipelinessa käytetään kanssa mvn package, joten frontti buildaantuu siellä sen yhteydessä. Ja react buildataan ja servataan siellä myös oikein.
  • Valitsin frontin sijainniksi src/main/frontend, koska frontend-maven-plugin - plugarin esimerkeissä tehdään niin. https://github.com/Opetushallitus/koski - repossa frontti on laitettu web - kansion alle. Sekin olisi mielestäni ihan järkevä sijainti frontille.

@saku-koodari saku-koodari requested a review from a team September 10, 2024 13:15
next-env.d.ts

# Generated
node
Copy link
Contributor Author

Choose a reason for hiding this comment

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

oon lisännyt tämän käsin, koska buildissa ladaaan node tähän kansioon.

@saku-koodari
Copy link
Contributor Author

Nyt kun mietin asiaa (ja hieman infra-pullaria vilkuilleena), ehkä meidän frontend kannattaisi olla repon juuressa. Pitäisin meidän tapauksessa sen kuitenkin frontend - nimellä, koska mun mielestä se kuvaa paremmin kansiota.

by pointing the distDir in the frontend app straigth to maven's target/classes/static - folder
@saku-koodari saku-koodari merged commit 510ca95 into main Sep 11, 2024
2 checks passed
@saku-koodari saku-koodari deleted the feat/react branch September 11, 2024 10:30
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