fschulze / sqlalchemy_schemadisplay

Other
140 stars 37 forks source link

Change column name appearance and allow user to restrict tables from schema #7

Closed cchrysostomou closed 6 years ago

cchrysostomou commented 6 years ago

1) This will add a (FK) suffix to column names that are foreign keys and (PK) suffix to column names that are primary keys. 2) in addition a user can add new argument to create_schema_graph where they pass in a list of table names they want rendered. Useful if schema has many tables and only want to show a subset of relationships.

fschulze commented 6 years ago

Thanks, looks useful. Could you please rebase with current master, add a changlog entry and fix the tests? They broke because of the additional suffix.

cchrysostomou commented 6 years ago

ok made the changes requested and the tests passed on my end. Adding FK/PK is now optional. I also added two more tests to confirm that FK/PK is added to the html when desired.

fschulze commented 6 years ago

I've rebased and merged