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

Førebu bibliotek, del 1: Kutte koplingar til spesifikke malar frå brevbaker #1072

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

Conversation

madsop-nav
Copy link
Contributor

@madsop-nav madsop-nav commented Nov 24, 2024

Legg til rette for å kunne gjera brevbaker til bibliotek ved å gå bort frå å bruke spesifikke pensjonsmalar i testar etc, og å starte å innføre abstraksjonslag og saumar for å kople inn dei spesifikke breva.

Forholdsvis ufarleg og enkel PR, som legg til rette for å gjera større endringar etterpå. Steg for steg.

@madsop-nav madsop-nav requested review from hanneanes, paomoo and a team as code owners November 24, 2024 18:01
@madsop-nav madsop-nav changed the title Foerebu bibliotek, del 1: Kutte koplingar til spesifikke malar frå brevbaker Førebu bibliotek, del 1: Kutte koplingar til spesifikke malar frå brevbaker Nov 24, 2024
…å i neste runde flytte ProductionTemplates ut frå repoet
… eiga fil, så resten av modul-initen kan vera uavhengig av lista over malar

configureMetrics()
brevbakerRouting(jwtConfigs.map { it.name }.toTypedArray(), latexCompilerService, templates)
monitor.subscribe(ServerReady) { it.log.info("Ferdig med å sette opp applikasjonen") }
Copy link
Contributor

Choose a reason for hiding this comment

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

Kan kanskje flytte denne subscriben opp til den andre i toppen av modulen?

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