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

Handle CORS #116

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Handle CORS #116

wants to merge 7 commits into from

Conversation

torleifg
Copy link

@torleifg torleifg commented Jan 28, 2025

Interessant testoppsett. Her er eit forsøk på å presse inn implementasjon og test av CORS. Opner ikkje pull request då det vel er behov for variabler i kubernetes .

@NilsOveTen
Copy link
Contributor

Interessant testoppsett.

😅
Rust-applikasjonene har i stor grad dårlig testing, tror ikke dette er det verste:P
Har behov for litt standardisering på hvordan sette opp tester i rust, om du orker ta tak i det så har vi jo dette issuet liggende her: #74

er dette han sikter til: https://github.com/Informasjonsforvaltning/fdk-mqa-scoring-api/blob/main/.github/workflows/deploy-staging.yaml#L10-L15
De blir ikke kjørt i workflow, så er sikkert noe som feiler

@torleifg
Copy link
Author

Interessant testoppsett.

😅 Rust-applikasjonene har i stor grad dårlig testing, tror ikke dette er det verste:P Har behov for litt standardisering på hvordan sette opp tester i rust, om du orker ta tak i det så har vi jo dette issuet liggende her: #74

er dette han sikter til: https://github.com/Informasjonsforvaltning/fdk-mqa-scoring-api/blob/main/.github/workflows/deploy-staging.yaml#L10-L15 De blir ikke kjørt i workflow, så er sikkert noe som feiler

Det blir ikkje fiksa i denne PR. Må i alle fal ha ein ferdig migrert postgres database køyrande for at testene skal lyse grønt, per i dag.

@torleifg torleifg marked this pull request as ready for review January 29, 2025 12:46
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.

3 participants