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

Attribute en Consumer tabel verplaatsen naar kolom niveau #52

Open
kevinhagenaars opened this issue Dec 6, 2023 · 1 comment
Open
Labels
enhancement New feature or request

Comments

@kevinhagenaars
Copy link
Collaborator

In de huidige situatie is er een tabel genaamd "Attributes" en "Consumers" welke een aantal optionele foreign keys bevat. Deze tabel dient nu als een verzamelbak voor alle attributen over alle entiteiten die dit implementeren.

Door dit te verplaatsen naar een kolom op de betreffende entiteiten verwijder je hiermee de relaties waardoor joins niet meer nodig zijn. In de betreffende kolom wordt een json array opgeslagen welke door EF Core wordt getransformeerd tot een concreet object (https://www.learnentityframeworkcore.com/misc/json-columns)

Hierdoor wordt de impact op de name en description mapping geminimaliseerd omdat er aan de structuur in code niets wijzigt.

Echter is dit alleen mogelijk in .NET 7 dus zal het project eerst geupdate moeten worden naar .NET 8 zoals beschreven in #51

@kevinhagenaars kevinhagenaars added the enhancement New feature or request label Dec 6, 2023
@kevinhagenaars
Copy link
Collaborator Author

Momenteel is de upgrade naar .NET 8 (#51) en deze wijziging gemaakt in de branch han-main. Echter zijn er nog problemen met deze mapping voor het mapping profile van RIO met betrekking tot het consumer attribuut. Wij hebben hierover contact met SURF en wachten even hun reactie af.

Op het moment dat wij meer info hebben en een eventuele fix hebben doorgevoerd op die branch, dan zullen we een PR open zetten om zowel #51 als deze fix door te voeren naar de main branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant