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

App: use dynamic inputs #18

Merged
merged 16 commits into from
May 20, 2024
Merged

App: use dynamic inputs #18

merged 16 commits into from
May 20, 2024

Conversation

TommyBom
Copy link
Collaborator

@TommyBom TommyBom commented May 17, 2024

  • dynamically load content for pruefi-select and formatversion-select
  • add selects to table page header
  • add GitHub ci to prevent formatter back and fourth

@TommyBom TommyBom marked this pull request as ready for review May 17, 2024 19:16
@TommyBom TommyBom requested review from ohenning and hf-krechan May 17, 2024 19:16
Comment on lines +42 to +46
- name: Build frontend
run: npm run ng:build

- name: Build server
run: npm run server:build
Copy link
Collaborator

Choose a reason for hiding this comment

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

these steps just test if the build process is working?

@@ -0,0 +1 @@
src/app/core/api
Copy link
Collaborator

Choose a reason for hiding this comment

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

why do we ignore them? Are these autogenerated files?

@TommyBom TommyBom merged commit 403838d into main May 20, 2024
1 check passed
@TommyBom TommyBom deleted the feature/dynamic-inputs branch May 20, 2024 09:34
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