-
Notifications
You must be signed in to change notification settings - Fork 41
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
Orschestrate view page issues #586
Comments
|
|
@fatchat i like the current fix with basic and advanced. Some small changes for now until users ask for more
@siddhant3030 ccould you create a separate issues for 4. 'edit custom dbt task' and 5. dropdown for select and exclude in custom dbt task |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
@nairabhishek73 we discussed multiple issues today so I'm mentioning it here
https://jam.dev/c/76589a6d-12fb-4fdd-94a4-3aae87428129
I wasn't able to select any of the dbt task when I first selected one of the pipeline to run. Only the custom one I can select.
I had to delete all the task which then comes to dropdown. This is shown in the video above
I don't know what numbering does here. I'm confused what should I do when I first see this screen. Should I type number 1 or delete all the tasks and then select one task to run. This looks super confusing and I think we dont need this. Why can't have we just dropdown and by default to "dbt run" or something.
System and Client Info I dont think we need it. We don;t need to explicitly call this. IMO
Why there is not edit option in the custom dbt task? I should be able to go back and edit custom functions
https://jam.dev/c/89e95c65-cbdf-42ce-8fcb-5d77f391323d
When we create a dbt function we should be able to select models from the dropdown. I mean select option mean that right? that user should be able to select models which he wants to run. Otherwise its confusing or we should rename it to "type your model" which you want to run
https://jam.dev/c/a7e4660d-24c8-49b7-bcf2-d833d6f60c82
@nairabhishek73 I think this all what we discussed today. Please look into this
The text was updated successfully, but these errors were encountered: