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

Revised dialog model #836

Closed
8 tasks done
Tracked by #397
elsand opened this issue Jun 12, 2024 · 0 comments
Closed
8 tasks done
Tracked by #397

Revised dialog model #836

elsand opened this issue Jun 12, 2024 · 0 comments
Labels
epic Groups several issues relating to some functionality

Comments

@elsand
Copy link
Member

elsand commented Jun 12, 2024

This is the meta-issue tracking the work for implementing the new dialog model. Current written draft is available at https://gist.github.com/elsand/ef205ad479b3377aea98506808d9a84f

This list is work-in-progress

@elsand elsand converted this from a draft issue Jun 12, 2024
@elsand elsand added the epic Groups several issues relating to some functionality label Jun 12, 2024
oskogstad added a commit that referenced this issue Jun 12, 2024
<!--- Provide a general summary of your changes in the Title above -->

## Description
Prep. for model change, DialogElements/Transmissions will be immutable
Removing PUT/DELETE REST endpoints

<!--- Describe your changes in detail -->

## Related Issue(s)

- #836

## Verification

- [x] **Your** code builds clean without any errors or warnings
- [x] Manual testing done (required)
- [x] Relevant automated test added (if you find this hard, leave it and
we'll help out)

## Documentation

- [ ] Documentation is updated (either in `docs`-directory, Altinnpedia
or a separate linked PR in
[altinn-studio-docs.](https://github.com/Altinn/altinn-studio-docs), if
applicable)
oskogstad added a commit that referenced this issue Jun 28, 2024
<!--- Provide a general summary of your changes in the Title above -->

## Description

<!--- Describe your changes in detail -->
Because of the new model changes, we no longer need to load all children
when changing the dialog aggregate.
We leave the code in case we need it in the future. (If sub entities
require their own deleted/updated events)

## Related Issue(s)

- #557 
- #836 

## Verification

- [ ] **Your** code builds clean without any errors or warnings
- [ ] Manual testing done (required)
- [ ] Relevant automated test added (if you find this hard, leave it and
we'll help out)

## Documentation

- [ ] Documentation is updated (either in `docs`-directory, Altinnpedia
or a separate linked PR in
[altinn-studio-docs.](https://github.com/Altinn/altinn-studio-docs), if
applicable)
@elsand elsand closed this as completed Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
epic Groups several issues relating to some functionality
Projects
None yet
Development

No branches or pull requests

1 participant