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(form-system-backend): Add functionality to create and edit custom lists #15352

Merged
merged 92 commits into from
Aug 9, 2024

Conversation

OlafurGeorg
Copy link
Contributor

@OlafurGeorg OlafurGeorg commented Jun 26, 2024

...

What

Add functionality to the backend to create and edit custom lists

Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • Formatting passes locally with my changes
  • I have rebased against main before asking for a review

Hyosam and others added 30 commits February 1, 2024 15:46
…cript, removed all console.logs and removed all code that was commented out
@datadog-island-is
Copy link

Datadog Report

All test runs 45d48da 🔗

4 Total Test Services: 0 Failed, 4 Passed
➡️ Test Sessions change in coverage: 7 no change

Test Services
Service Name Failed Known Flaky New Flaky Passed Skipped Total Time Code Coverage Change Test Service View
judicial-system-api 0 0 0 54 0 6.92s 1 no change Link
judicial-system-formatters 0 0 0 36 0 6.09s 1 no change Link
judicial-system-message 0 0 0 30 0 12.38s 1 no change Link
judicial-system-message-handler 0 0 0 3 0 4.7s 1 no change Link

Copy link

codecov bot commented Jul 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 37.01%. Comparing base (61ba7e9) to head (eb4cc5e).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #15352      +/-   ##
==========================================
+ Coverage   36.97%   37.01%   +0.03%     
==========================================
  Files        6567     6560       -7     
  Lines      134328   134206     -122     
  Branches    38429    38390      -39     
==========================================
+ Hits        49668    49671       +3     
+ Misses      84660    84535     -125     
Flag Coverage Δ
web 1.89% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 82 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 61ba7e9...eb4cc5e. Read the comment docs.

@baering
Copy link
Member

baering commented Jul 18, 2024

There's still a missing infra definition for the service, you can consult with the devops team on that.

@baering baering requested a review from a team as a code owner July 18, 2024 11:41
Copy link
Member

@AndesKrrrrrrrrrrr AndesKrrrrrrrrrrr left a comment

Choose a reason for hiding this comment

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

Infra LGTM

@OlafurGeorg OlafurGeorg added the automerge Merge this PR as soon as all checks pass label Aug 9, 2024
@kodiakhq kodiakhq bot merged commit f75cac3 into main Aug 9, 2024
40 checks passed
@kodiakhq kodiakhq bot deleted the form-system-backend-client branch August 9, 2024 13:04
oskarjs pushed a commit that referenced this pull request Aug 20, 2024
…m lists (#15352)

* form builder 'init'

* Added functionality for relevant parties

* redirect to the correct url when creating a new form

* removed secret from comment

* chore: nx format:write update dirty files

* Property number component

* Fixed the header

* Reverted to next 13.3.0

* Deleted the e2e project, updated to project to adhere to strict typescript, removed all console.logs and removed all code that was commented out

* chore: nx format:write update dirty files

* Started implementing zod validation

* chore: nx format:write update dirty files

* Generated a nestjs client from our openapi schema

* Generated a client and created a domain client for graphql

* Update Form, Group, Input, and Step DTOs

* Refactor input settings in FormsService and Input model

* Update API and authentication configurations

* Added form-system client and module to the app/api and added a codegen + script to generate graphql schemas

* Remove form-system module and add formSystemNavigation to masterNavigation

* Refactor form-system API code and formBuilder into portals/admin

* Fixng typing erros in form system graphql domain

* Added another Object type and a bunch of inputTypes in order for codegen to work

* Update import statements for LanguageTypeInput

* Update form system module and related files

* Update FormSystemClient.config and Forms component

* Saving progress

* Working version. Current issue: inputsettings are not being saved due to graphql not handling type discriminators

* took out the formsysteminputsettings and replaced it with a graphqlTypeJson

* spring cleaning and creating dependencies within the form

* removed form-builder app

* chore: charts update dirty files

* chore: nx format:write update dirty files

* form-system backend initial commit

* initial configuration

* adjust for PR

* fix lint warnings

* add models and modules

* helstu módel, services og controllerar

* make lint satisfied

* Add functionality to create and edit custom lists

* application part start

* change step to section and group to page

* tiny lint things

* change naming of page to screen

* changed naming of input to field

* fix lint warnings

* change naming of input to field

* work on application part

* change tablenames to singular

* changed naming of testimony to certification

* fixing apiproperty configurations

* changed naming of urlName to slug

* introduce lodash pick

* refactor vegna PR

* add openapi.yaml build

* Refactor project setup

* Add class validation for CreateFormDto

* add class-validation

* use predefined types for cases in switch

* Fix build

* Fix lint

* Update tags

* Remove empty test files

* Skip tests until implemented

* Add advania as codeowner for new service

* add infra definition

* add the service to uber-charts

* chore: charts update dirty files

* update grantNamespaces

* chore: charts update dirty files

* added seed instruction to DLS

* chore: charts update dirty files

---------

Co-authored-by: Hyo-Sam Nandkisore <[email protected]>
Co-authored-by: andes-it <[email protected]>
Co-authored-by: Sævar Már Atlason <[email protected]>
Co-authored-by: Bæring Gunnar Steinþórsson <[email protected]>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
jonnigs pushed a commit that referenced this pull request Sep 2, 2024
…m lists (#15352)

* form builder 'init'

* Added functionality for relevant parties

* redirect to the correct url when creating a new form

* removed secret from comment

* chore: nx format:write update dirty files

* Property number component

* Fixed the header

* Reverted to next 13.3.0

* Deleted the e2e project, updated to project to adhere to strict typescript, removed all console.logs and removed all code that was commented out

* chore: nx format:write update dirty files

* Started implementing zod validation

* chore: nx format:write update dirty files

* Generated a nestjs client from our openapi schema

* Generated a client and created a domain client for graphql

* Update Form, Group, Input, and Step DTOs

* Refactor input settings in FormsService and Input model

* Update API and authentication configurations

* Added form-system client and module to the app/api and added a codegen + script to generate graphql schemas

* Remove form-system module and add formSystemNavigation to masterNavigation

* Refactor form-system API code and formBuilder into portals/admin

* Fixng typing erros in form system graphql domain

* Added another Object type and a bunch of inputTypes in order for codegen to work

* Update import statements for LanguageTypeInput

* Update form system module and related files

* Update FormSystemClient.config and Forms component

* Saving progress

* Working version. Current issue: inputsettings are not being saved due to graphql not handling type discriminators

* took out the formsysteminputsettings and replaced it with a graphqlTypeJson

* spring cleaning and creating dependencies within the form

* removed form-builder app

* chore: charts update dirty files

* chore: nx format:write update dirty files

* form-system backend initial commit

* initial configuration

* adjust for PR

* fix lint warnings

* add models and modules

* helstu módel, services og controllerar

* make lint satisfied

* Add functionality to create and edit custom lists

* application part start

* change step to section and group to page

* tiny lint things

* change naming of page to screen

* changed naming of input to field

* fix lint warnings

* change naming of input to field

* work on application part

* change tablenames to singular

* changed naming of testimony to certification

* fixing apiproperty configurations

* changed naming of urlName to slug

* introduce lodash pick

* refactor vegna PR

* add openapi.yaml build

* Refactor project setup

* Add class validation for CreateFormDto

* add class-validation

* use predefined types for cases in switch

* Fix build

* Fix lint

* Update tags

* Remove empty test files

* Skip tests until implemented

* Add advania as codeowner for new service

* add infra definition

* add the service to uber-charts

* chore: charts update dirty files

* update grantNamespaces

* chore: charts update dirty files

* added seed instruction to DLS

* chore: charts update dirty files

---------

Co-authored-by: Hyo-Sam Nandkisore <[email protected]>
Co-authored-by: andes-it <[email protected]>
Co-authored-by: Sævar Már Atlason <[email protected]>
Co-authored-by: Bæring Gunnar Steinþórsson <[email protected]>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge this PR as soon as all checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants