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

Feature/transfer tool #917

Merged
merged 9 commits into from
Jul 28, 2023
Merged

Feature/transfer tool #917

merged 9 commits into from
Jul 28, 2023

Conversation

hvilander
Copy link
Contributor

@hvilander hvilander commented Jul 26, 2023

This has been reviewed and I am looking to merge this feature branch into master.

This adds a new dialog to provide a data driven way to interact with products.

* feat: 🎸 SQFormTransferTool

Creates a new component SQFormTransferTool. It is an opinionated, data
driven modal to aid in call transfers

* chore: 🤖 Fixes test.ci run

jest requires relative paths with our current config

* chore: 🤖 Type fix and TODO Comment added

* chore: 🤖 Adds title prop to transfer tool
* feat: 🎸 SQFormTransferTool

Creates a new component SQFormTransferTool. It is an opinionated, data
driven modal to aid in call transfers

* chore: 🤖 Type fix and TODO Comment added

* feat: 🎸 Sections in side of the accordions
* feat: 🎸 Step Rendering

Render the steps for each product

* chore: 🤖 Fixes some imports that broke some tests
Given that every field is dropdown with provided options, every field is
already valid
* feat: 🎸 adds the viewed products ids to callbacks

* style: 💄 Clean up the loading view styles (#912)
Copy link
Contributor

@20BBrown14 20BBrown14 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#yolo

@hvilander hvilander merged commit 71426ef into master Jul 28, 2023
1 check passed
@hvilander hvilander deleted the feature/TransferTool branch July 28, 2023 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants