Skip to content

New: SD-966-Live-Chat#13443

Open
nidillon wants to merge 4 commits intolinode:developfrom
nidillon:SD-966
Open

New: SD-966-Live-Chat#13443
nidillon wants to merge 4 commits intolinode:developfrom
nidillon:SD-966

Conversation

@nidillon
Copy link
Contributor

Description 📝

Highlight the Pull Request's context and intentions.

Changes 🔄

  • Adding livechat features on General/Accounts/Billing
    List any change(s) relevant to the reviewer.
  • ...
  • ...

Scope 🚢

Upon production release, changes in this PR will be visible to:

  • All customers
  • Some customers (e.g. in Beta or Limited Availability)
  • No customers / Not applicable

Target release date 🗓️

Please specify a release date (and environment, if applicable) to guarantee timely review of this PR. If exact date is not known, please approximate and update it as needed.

Preview 📷

Include a screenshot <img src="" /> or video <video src="" /> of the change.

🔒 Use the Mask Sensitive Data setting for security.

💡 For changes requiring multiple steps to validate, prefer a video for clarity.

Before After

|
image
|
image
|

How to test 🧪

Prerequisites

(How to setup test environment)

  • ...
  • ...

Reproduction steps

(How to reproduce the issue, if applicable)

  • To test navigate to the "Help and Support" tab as you normally would use to open a ticket
  • Next click General/Account/Billing and it should show "Start a Live Chat" instead of showing Open a ticket.
  • Once a title is entered you will be able to proceed with a "Start A Live Chat"
  • Once "Start a live Chat is started you will see a window terminal start where it would allow it to grab the title and the provided token to verify they are a real person.

Verification steps

(How to verify changes)

  • ...
  • ...
Author Checklists

As an Author, to speed up the review process, I considered 🤔

👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support


  • I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

  • All tests and CI checks are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 9 failing tests on test run #1 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
9 Failing860 Passing11 Skipped52m 31s

Details

Failing Tests
SpecTest
smoke-linode-landing-table.spec.tsCloud Manager Cypress Tests→linode landing checks » checks the landing page side menu items
smoke-linode-landing-table.spec.tsCloud Manager Cypress Tests→linode landing checks » checks the create menu dropdown items
open-support-ticket.spec.tsCloud Manager Cypress Tests→open support tickets » can open a support ticket
open-support-ticket.spec.tsCloud Manager Cypress Tests→open support tickets » can create a ticket with a severity level specified
open-support-ticket.spec.tsCloud Manager Cypress Tests→open support tickets » can create a support ticket with an entity
alerting-notification-channel-permission-tests.spec.tsCloud Manager Cypress Tests→Notification Channel Listing Page — Access Control » allows access when notificationChannels is enabled
alerting-notification-channel-permission-tests.spec.tsCloud Manager Cypress Tests→Notification Channel Listing Page — Access Control » hides the Notification Channels tab when notificationChannels is disabled
account-switching.spec.tsCloud Manager Cypress Tests→Parent/Child account switching→From Parent to Child » can search child accounts
account-switching.spec.tsCloud Manager Cypress Tests→Parent/Child account switching→From Child to Child » can switch to another Child account as a Proxy user

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/linodes/smoke-linode-landing-table.spec.ts,cypress/e2e/core/helpAndSupport/open-support-ticket.spec.ts,cypress/e2e/core/cloudpulse/alerting-notification-channel-permission-tests.spec.ts,cypress/e2e/core/parentChild/account-switching.spec.ts"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Review

Development

Successfully merging this pull request may close these issues.

2 participants