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

4. Update the Transform Section of the DDP UI #107

Closed
nairabhishek73 opened this issue Jun 15, 2023 · 3 comments · Fixed by #194
Closed

4. Update the Transform Section of the DDP UI #107

nairabhishek73 opened this issue Jun 15, 2023 · 3 comments · Fixed by #194
Assignees

Comments

@nairabhishek73
Copy link

Describe the task

  1. Add the run option
  2. Add 3 clickable options
  • Run
  • Other actions(clean, deps, test, docs-generate)-
  • options/edit 3 dots
  1. Keep one logs box for all DBT transform actions

Expected behaviour
All formatting should match that of Figma. With icons font characteristica, colour palette, etc.
All changes should be made as listed above.

References
DDP UI
Figma
UI feedback working doc

@mdshamoon

@nairabhishek73 nairabhishek73 changed the title 1. Update the Transform Section of the DDP UI 4. Update the Transform Section of the DDP UI Jun 15, 2023
@nairabhishek73
Copy link
Author

3 dots are not working. Keeping this comment here as I don't see another issue created.

For a new login adding their first transformation, it should say "Connect to DBT repository", not "Connect Repo"

@fatchat
Copy link
Contributor

fatchat commented Jun 28, 2023

Linked to C4GT Community issue #162 for the remaining tasks

@fatchat
Copy link
Contributor

fatchat commented Jul 13, 2023

Edit DBT repo url + target schema is pending

@fatchat fatchat self-assigned this Jul 13, 2023
@fatchat fatchat linked a pull request Jul 15, 2023 that will close this issue
@fatchat fatchat closed this as completed Jul 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants