Closed PiJoCoder closed 10 months ago
@hacitandogan, would you like to think about this one next. The request here is to expose the actual query behind the report in case people want to use it for direct analysis in SSMS. I can demo if you'd like
@PiJoCoder , sure I will check this one and ping you for the demo and sync.
@PiJoCoder , sure I will check this one and ping you for the demo and sync.
You can use logic similar to this (in server configuration report), where the text of the query is not exposed at first, but we can have some toggle +/- to show it
@PiJoCoder
I did a sample on one of the reports to test this.
This is server configuration report , which has multiple datasets;
Hidden:
Visible:
Tried to keep it plain and simple but should we highlight that more or is this good ? Once we clarify how it should look , my plan is to use same approach on the other reports as well.
@PiJoCoder
I did a sample on one of the reports to test this.
This is server configuration report , which has multiple datasets;
Hidden:
Visible:
Tried to keep it plain and simple but should we highlight that more or is this good ? Once we clarify how it should look , my plan is to use same approach on the other reports as well.
I agree with you that plain and simple is good. I don't think we need to go fancy on this. One thing that I image is to add this at the bottom of reports, away from other report data so people don't get confused. Is that how you image it too?
@PiJoCoder , I sent a PR for one report only so you can also see and test this. If we are OK with this , I can implement the same for all reports.
@PiJoCoder , I sent a PR for one report only so you can also see and test this. If we are OK with this , I can implement the same for all reports.
Thank you! I'll test and let you know
@PiJoCoder , I sent a PR for one report only so you can also see and test this. If we are OK with this , I can implement the same for all reports.
Looks great, @hacitandogan! Let's proceed with it. Thank you
PR merged , closing this item.