Skip to content

fix(frontend): handle data modal context correctly to update after deleting entries#984

Merged
tom-rm-meyer-ISST merged 3 commits intoeclipse-tractusx:mainfrom
achtzig20:fix/data-modal-context-bug
Sep 26, 2025
Merged

fix(frontend): handle data modal context correctly to update after deleting entries#984
tom-rm-meyer-ISST merged 3 commits intoeclipse-tractusx:mainfrom
achtzig20:fix/data-modal-context-bug

Conversation

@OlgaIvkovic
Copy link
Copy Markdown
Contributor

@OlgaIvkovic OlgaIvkovic commented Sep 23, 2025

Description

  • Fixed modal table not updating after deleting an entry

Solves #978

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

  • DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
  • Copyright and license header are present on all affected files
  • If helm chart has been changed, the chart version has been bumped to either next major, minor or patch level (compared to released chart).

Copy link
Copy Markdown
Contributor

@tom-rm-meyer-ISST tom-rm-meyer-ISST left a comment

Choose a reason for hiding this comment

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

Thanks a lot! Works great, please update changelog.

Comment thread CHANGELOG.md Outdated
* Updated open API ([#929](https://github.com/eclipse-tractusx/puris/pull/929))
* Updated dependencies to resolve high and critical security vulnerabilities ([#932](https://github.com/eclipse-tractusx/puris/pull/932))

### Fixes
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please rebase and add to v3.4.1 that has not yet been released.

@tom-rm-meyer-ISST tom-rm-meyer-ISST linked an issue Sep 24, 2025 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

@tom-rm-meyer-ISST tom-rm-meyer-ISST left a comment

Choose a reason for hiding this comment

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

LGTM, thanks a lot!

@tom-rm-meyer-ISST tom-rm-meyer-ISST changed the title Fix/data modal context bug fix(frontend): handle data modal context correctly to update after deleting entries Sep 26, 2025
@tom-rm-meyer-ISST tom-rm-meyer-ISST merged commit 25cca6d into eclipse-tractusx:main Sep 26, 2025
13 checks passed
tom-rm-meyer-ISST pushed a commit that referenced this pull request Oct 9, 2025
…988)

* fix/bruno dtr setup (#967)

* fix: dtr framework policy in bruno

* fix: added missing access policy namespaces

* chore: updated changelog

* fix: revert policy creation fix to preserve existing policy definitions

* fix(frontend): handle data modal context correctly to update after deleting entries (#984)

* fixed the tables not updating after deleting rows in different modals of the dataModalContext

* chore: update changelog

* chore: moved to the correct part of changelog

* feat: implemented multi profile support for backwards compatibility

* fix: properly separated JsonLd context by profile

* fix: minor improvements

* chore: add new jsonld files to remote contexts in notice

---------

Co-authored-by: OlgaIvkovic <olga.ivkovic@8020.eco>
ReneSchroederLJ added a commit to achtzig20/puris that referenced this pull request Mar 3, 2026
…clipse-tractusx#988)

* fix/bruno dtr setup (eclipse-tractusx#967)

* fix: dtr framework policy in bruno

* fix: added missing access policy namespaces

* chore: updated changelog

* fix: revert policy creation fix to preserve existing policy definitions

* fix(frontend): handle data modal context correctly to update after deleting entries (eclipse-tractusx#984)

* fixed the tables not updating after deleting rows in different modals of the dataModalContext

* chore: update changelog

* chore: moved to the correct part of changelog

* feat: implemented multi profile support for backwards compatibility

* fix: properly separated JsonLd context by profile

* fix: minor improvements

* chore: add new jsonld files to remote contexts in notice

---------

Co-authored-by: OlgaIvkovic <olga.ivkovic@8020.eco>
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.

[Bug] Modal dialogs are not updated when deleting entries

2 participants