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

Display provider-specific logo for Azure data-planes #1497

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kiahna-tucker
Copy link
Member

@kiahna-tucker kiahna-tucker commented Mar 6, 2025

Issues

The issues directly below are completely resolved by this PR:
#1494

Changes

1494

The following features are included in this PR:

  • Display the data-plane provider-specific logo for Azure data-planes where applicable.

  • Extend unit test coverage for data-plane utils.

Tests

Manually tested

Approaches to testing are as follows:

  • Validate that the Azure logo renders as expected in the data-plane selector.

Automated tests

Test coverage extended for the following utils:

  • formatDataPlaneName

  • parseDataPlaneName

Playwright tests ran locally

  • Admin
  • Captures
  • Collections
  • HomePage
  • Login
  • Materialization

Screenshots

Light Mode

Data-plane selector | Menu options

pr_screenshot-1497-azure_data_plane-selector-menu_options

Data-plane selector | Public, Azure data-plane selected

pr_screenshot-1497-azure_data_plane-selector-public_selected

Data-plane selector | Private, Azure data-plane selected

pr_screenshot-1497-azure_data_plane-selector-private_selected

Dark Mode

Data-plane selector | Menu options

pr_screenshot-1497-azure_data_plane-selector-menu_options-dark

Data-plane selector | Public, Azure data-plane selected

pr_screenshot-1497-azure_data_plane-selector-public_selected-dark

Data-plane selector | Private, Azure data-plane selected

pr_screenshot-1497-azure_data_plane-selector-private_selected-dark

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.

1 participant