Geido

Results 138 comments of Geido

@justinpark I am still not fully convinced of the schema dropdown on top of the editor as I am concerned that it creates the impression that the SQL is strictly...

Hello @puridach-w, feel free to ping me when you are ready for another round of review. Thanks!

/testenv up FEATURE_ALERT_REPORTS=true

Np @puridach-w. There are a few conflicts as I just merged a PR that changes the styles of the A&R modal. I hope that does not become too much of...

@puridach-w just messaged you on Apache Superset Slack to help with the CI issues

/testenv up FEATURE_ALERT_REPORTS=true

Hey @puridach-w doing some manual testing. It seems that the spacing is off here ![Screenshot 2024-03-11 at 16 41 35](https://github.com/apache/superset/assets/60598000/0cb4cb82-3a51-4712-8eab-31866e8479c6)

/testenv up FEATURE_ALERT_REPORTS=true

Hey @puridach-w if you look at the testenv above, you will see that there is a` :` in the placeholder. I think something might be broken in the placeholder logics....