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

More Best practices rules+ SQL Nexus main report new look #220

Merged

Conversation

PiJoCoder
Copy link
Contributor

No description provided.

@PiJoCoder
Copy link
Contributor Author

PiJoCoder commented Feb 3, 2023

To test #217

Just explore the new design of the SQL Nexus main form

image

To test #216

To test, try to import data in an existing SQL Nexus db that has prior data imported into it. You will be told that there is already data but the Import diaglog will not close like it used to do.

To test #215

This may be a bit challenging to test unless you re-created the conditions which hit these rules or manually engineer the raw data in the output files to match the conditions. But if you have feedback on output - here are screenshots

image

image

image

image

To test #210

Explore the database properties report

image

image

image

@PiJoCoder
Copy link
Contributor Author

Additional VLF report

image

image

asavioliMSFT
asavioliMSFT previously approved these changes Feb 9, 2023
Copy link
Contributor

@asavioliMSFT asavioliMSFT left a comment

Choose a reason for hiding this comment

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

Approved after discussing with Joseph

and minor cosmetic change in what we print
@PiJoCoder PiJoCoder merged commit f1625f6 into master Feb 10, 2023
@PiJoCoder PiJoCoder deleted the BestPracticesRules+DbPropertiesReport_pijocoder_012023 branch February 10, 2023 03:09
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