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

feature/model update v3 #187

Merged
merged 3 commits into from
Oct 31, 2024
Merged

feature/model update v3 #187

merged 3 commits into from
Oct 31, 2024

Conversation

cutoffthetop
Copy link
Contributor

@cutoffthetop cutoffthetop commented Oct 28, 2024

Changes

  • upgrade mex-common and mex-model dependencies to metadata model v3
  • apply additional ruff linters in prep for new ruff config

Copy link
Contributor

@mr-kamran-ali mr-kamran-ali left a comment

Choose a reason for hiding this comment

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

nice and clean. 🚀

@cutoffthetop cutoffthetop merged commit 8e640f4 into main Oct 31, 2024
4 checks passed
@cutoffthetop cutoffthetop deleted the feature/model-update-v3 branch October 31, 2024 12:07
rababerladuseladim added a commit to robert-koch-institut/mex-editor that referenced this pull request Nov 14, 2024
# PR Context
- this PR only resolves Links, Texts and Enums, but does not resolve
Identifiers yet
- additionally cleans up the user interface and code structure a bit
- needs robert-koch-institut/mex-backend#187 and
robert-koch-institut/mex-backend#189

# Changes
- upgrade mex-common and model dependencies to v3
- overhaul and simplify margins and spaces
- move transform_models_to_fields from State to transform module
- use dedicated backend connector methods for edit and search
- use same rendering components for edit and search pages
- pop toasts when backend connector encounters errors
- scroll to top when pagination is triggered

# Fixed
- fix routing issues by moving the refresh handlers section.from
on_mount to page.on_load

---------

Signed-off-by: Nicolas Drebenstedt <[email protected]>
Co-authored-by: Franziska Diehr <[email protected]>
Co-authored-by: RKI | Metadata Exchange <[email protected]>
Co-authored-by: rababerladuseladim <[email protected]>
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