search

R0tenur / visualization

MIT License
170 stars 25 forks source link

Wrong relation direction #138

Open maxfaa opened 7 months ago

maxfaa commented 7 months ago

The result provided by the extension isn't correct. It doesn't match my teacher's schema.

Expected Behavior

image

Current Behavior

image

Steps to Reproduce

  1. Use the scripts in the zip below scripts-supplied-by-my-teacher.zip
  2. Run these
  3. Open the database schema visualization

Context

First, thank you so much. I'm a computer science student and in one of my courses I have to use SSMS. Unfortunately I'm a MacOS user (and one of my friends use Ubuntu). Since SSMS isn't available of ours OSs, we use Azure Data Studio. So thanks a lot for creating this extension that counters the lack of schema virtualisation feature.

However, we have a big issue: the schemas provided by your extension are kinda reversed. Like, if we look (screenshots above) at the relation between the "commande" and "employe" tables, we can see this:

Screenshot 2023-12-18 at 15 57 20

while it should look like this:

Screenshot 2023-12-18 at 15 55 03
jmanghani commented 6 months ago

Yup, have the same problem here.

psoszynski commented 2 months ago

Still an issue. Have the same problem. Please fix.

maxfaa commented 2 months ago

@R0tenur @bruce-dunwiddie ??

R0tenur commented 2 months ago

I am currently not activly working on this project. Feel free to open a PR

Issue is probably here: https://github.com/R0tenur/visualization/blob/master/backend%2Fsrc%2Fservices%2Fbuilder.service.ts#L49

And should be clear how to run in locally in the readme.