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

Connect datatypes with components #817

Merged
merged 247 commits into from
Jul 2, 2021
Merged

Conversation

TheSlimvReal
Copy link
Collaborator

@TheSlimvReal TheSlimvReal commented May 13, 2021

see issue: #764 #552 #180 #826

Visible/Frontend Changes

--

Architectural/Backend Changes

  • EntitySchema holds information how an attribute can be displayed
  • EntitySchema holds information how an attribute can be edited
  • EntitySchemaDatatype provides a default component for displaying an data type
  • EntitySchemaDatatype provides a default component for editing an data type
  • EntityListComponent automatically fetches column information from schema if no other definition is found
  • FormComponent automatically fetches form field information from schema if no other definition is found
  • EntitySubrecord automatically fetches display and edit information from schema if no other definition is found
  • EntityList uses EntitySubrecord
  • Automatically create filters for properties using schema information
  • Update documentation
  • Automatically create (popup) form for entities -> Automatic PopUps #834

@github-actions
Copy link
Contributor

sleidig and others added 20 commits June 30, 2021 17:41
# Conflicts:
#	src/app/child-dev-project/educational-material/model/materials.ts
#	src/app/core/config/config-migration.service.spec.ts
#	src/app/core/config/config-migration.service.ts
#	src/app/core/entity-components/entity-form/entity-form.service.spec.ts
#	src/app/core/entity-components/entity-form/entity-form.service.ts
#	src/app/core/entity-components/entity-form/entity-form/entity-form.component.ts
#	src/app/core/entity-components/entity-form/entity-form/entity-form.stories.ts
#	src/app/core/entity-components/entity-list/entity-list.component.ts
@ghost
Copy link

ghost commented Jul 2, 2021

DeepCode's analysis on #68d7ab found:

  • ⚠️ 2 warnings, ℹ️ 2 minor issues. 👇
  • ✔️ 3 issues were fixed.

Top issues

Description Example fixes
The usage of regex in replace will change only the first match. Consider using a regular expression with the global replacement flag (/g) to replace all occurrences. Occurrences: 🔧 Example fixes
Be aware that month indices used in Date are 0-based (0 = January, 1 = February, ...). Occurrences: 🔧 Example fixes
Use an array literal instead of the Array constructor. Occurrences: 🔧 Example fixes

👉 View analysis in DeepCode’s Dashboard | Configure the bot

👉 The DeepCode service and API will be deprecated in August, 2021. Here is the information how to migrate. Thank you for using DeepCode 🙏 ❤️ !

If you are using our plugins, you might be interested in their successors: Snyk's JetBrains plugin and Snyk's VS Code plugin.

Copy link
Member

@sleidig sleidig left a comment

Choose a reason for hiding this comment

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

The problem with the entity-select is actually independent of this PR, also occuring in master (--> #872)

Looks good otherwise now!

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 2, 2021

SonarCloud Quality Gate failed.

Bug D 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 22 Code Smells

No Coverage information No Coverage information
3.5% 3.5% Duplication

@sleidig sleidig merged commit 4320781 into master Jul 2, 2021
@sleidig sleidig deleted the connect-datatypes-with-components branch July 2, 2021 22:13
aam-digital-ci pushed a commit that referenced this pull request Jul 2, 2021
# [2.52.0](2.51.1...2.52.0) (2021-07-02)

### Features

* connect datatypes with components to simplify config ([#817](#817)) ([4320781](4320781)), closes [#764](#764) [#552](#552) [#180](#180) [#826](#826) [#834](#834)
@aam-digital-ci
Copy link
Collaborator

🎉 This PR is included in version 2.52.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

3 participants