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

fix(callbox): correct color on voice chat icon #52

Merged
merged 1 commit into from
Dec 13, 2023

Conversation

braddialpad
Copy link
Contributor

Description

The voice chat icon color on the new designs was supposed to be purple but it was set as green. Correcting this, will require a small change in product as well due to the slot in callbox.

Pull Request Checklist

  • Ask the contributors if a similar effort is already in process or has been solved.
  • Review the contribution guidelines.
  • Use staging as your pull request's base branch. (All PRs using production as its base branch will be declined).
  • Ensure all gulp scripts successfully compile.
  • Update, remove, or extend all affected documentation.
  • Ensure no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).

Obligatory GIF (super important!)

@braddialpad braddialpad self-assigned this Dec 13, 2023
@braddialpad braddialpad added the visual-test-ready Add this tag when the PR is ready for visual test, to trigger GHA visual tests label Dec 13, 2023
Copy link

✔️ Dialtone Deploy Preview ready!
😎 Browse the preview: https://dialtone.dialpad.com/deploy-previews/pr-52/

Copy link

✔️ Dialtone-vue 2 Deploy Preview ready!
😎 Browse the preview: https://dialtone.dialpad.com/vue/deploy-previews/pr-52/

Copy link

✔️ Dialtone-vue 3 Deploy Preview ready!
😎 Browse the preview: https://dialtone.dialpad.com/vue3/deploy-previews/pr-52/

@iropolo
Copy link
Contributor

iropolo commented Dec 13, 2023

Correcting this, will require a small change in product as well due to the slot in callbox.

How will be solve this?, will create a ticket or..?

@braddialpad
Copy link
Contributor Author

How will be solve this?, will create a ticket or..?

Yep there is already one created by QA. I will do the fix.

@braddialpad braddialpad merged commit 50defaf into staging Dec 13, 2023
10 of 11 checks passed
@braddialpad braddialpad deleted the fix/voice-chat-color branch December 13, 2023 19:24
juliodialpad pushed a commit that referenced this pull request Dec 13, 2023
## [2.106.2](dialtone-vue2/v2.106.1...dialtone-vue2/v2.106.2) (2023-12-13)

### Bug Fixes

* **Callbox:** correct color on voice chat icon ([#52](#52)) ([50defaf](50defaf))
* global lint ([#47](#47)) ([8148de3](8148de3))
* linter errors, add global lint command ([7ff09ea](7ff09ea))

### Documentation

* fix links, fix outdated documentation. ([#40](#40)) ([124082e](124082e))
Copy link

🎉 This PR is included in version 3.98.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released visual-test-ready Add this tag when the PR is ready for visual test, to trigger GHA visual tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants