Re: Lazy Consensus Proposal: Enable option for render HTML on Superset SQL Lab's result table

2024-03-20 Thread Treesak, Yanisa (Agoda)
@superset.apache.org Subject: Re: Lazy Consensus Proposal: Enable option for render HTML on Superset SQL Lab's result table Email received from the internet. If in doubt, don't click any link nor open any attachment ! This feature will be really helpful. Since

Re: Lazy Consensus Proposal: Enable option for render HTML on Superset SQL Lab's result table

2024-03-16 Thread Elizabeth Thompson
This feature will be really helpful. Since the functionality applies to the sql results lower down the page, would it make sense to put the toggle in the popover controls where the parameters, etc selections are? > On Mar 14, 2024, at 2:50 PM, Evan Rusackas wrote: > > Indeed, I understand

Re: Lazy Consensus Proposal: Enable option for render HTML on Superset SQL Lab's result table

2024-03-14 Thread Evan Rusackas
Indeed, I understand that the proposal is to render HTML by default (as it is today) and optionally disable it. This would mean it’s non-breaking, and adds functionality rather than removing it. Evan Rusackas On Mar 14, 2024 at 2:28 AM -0600, Carlos Alonso Vega , wrote: > Hi all, > >  Hope it

Re: Lazy Consensus Proposal: Enable option for render HTML on Superset SQL Lab's result table

2024-03-14 Thread Carlos Alonso Vega
Hi all,  Hope it is not late to add an opinion. In our case HTML rendering is mandatory in charts and dashboards. Maybe in SQLab could be optional or disabled (it could make sense to see actual data for debuging queries, and rendered result could not be so necessary at that level.) We show

Re: Lazy Consensus Proposal: Enable option for render HTML on Superset SQL Lab's result table

2024-03-13 Thread Evan Rusackas
Hi all, I think we’re all caught up and on the same page here now. In short: • HTML would be rendered by default • This switch would effectively disable the rendering If that’s all it does,, it sounds like there are no objections. Maybe it was just a matter of wording, where it would be “Add

Re: Lazy Consensus Proposal: Enable option for render HTML on Superset SQL Lab's result table

2024-03-08 Thread Maxime Beauchemin
A github discussion (as opposed to a SIP) may be sufficient here, but open to either. Clearly there's lots to talk about around the topic of supporting html. I did some work around bringing in more support and consistent support across the board in the past, mostly for links (``) in various table

Re: Lazy Consensus Proposal: Enable option for render HTML on Superset SQL Lab's result table

2024-03-08 Thread Evan Rusackas
On a related note, I just came across this old issue requesting such a feature. I’m going to move it to a Discussion since it’s not a bug, and we can close it out if/when this feature expands beyond SQL Lab. Feel free to add comments if you feel it’s warranted: 

Re: Lazy Consensus Proposal: Enable option for render HTML on Superset SQL Lab's result table

2024-03-08 Thread Michael S. Molina
It looks like I misunderstood the proposal. I was with the impression that we wanted to render the Google page inside SQL Lab but the proposal is just to switch the rendering of the result as HTML or plain text. If I understood correctly this time, then I have no objections. Best regards,