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

merging latest changes from linode:develop #40

Conversation

santoshp210-akamai
Copy link
Owner

Description πŸ“

Highlight the Pull Request's context and intentions.

Changes πŸ”„

List any change(s) relevant to the reviewer.

  • ...
  • ...

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 or screen recording of the change.

πŸ”’ Use the Mask Sensitive Data setting for security.

πŸ’‘ Use <video src="" /> tag when including recordings in table.

Before After
πŸ“· πŸ“·

How to test πŸ§ͺ

Prerequisites

(How to setup test environment)

  • ...
  • ...

Reproduction steps

(How to reproduce the issue, if applicable)

  • ...
  • ...

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 unit tests are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

Commit message and pull request title format standards

Note: Remove this section before opening the pull request
Make sure your PR title and commit message on squash and merge are as shown below

<commit type>: [JIRA-ticket-number] - <description>

Commit Types:

  • feat: New feature for the user (not a part of the code, or ci, ...).
  • fix: Bugfix for the user (not a fix to build something, ...).
  • change: Modifying an existing visual UI instance. Such as a component or a feature.
  • refactor: Restructuring existing code without changing its external behavior or visual UI. Typically to improve readability, maintainability, and performance.
  • test: New tests or changes to existing tests. Does not change the production code.
  • upcoming: A new feature that is in progress, not visible to users yet, and usually behind a feature flag.

Example: feat: [M3-1234] - Allow user to view their login history


dennisvankekem and others added 30 commits November 22, 2024 17:03
fix: [APL-417] - changed APL name to Akamai App Platform
Co-authored-by: Banks Nussman <115251059+bnussman-akamai@users.noreply.github.com>
Release v1.132.2 - `staging` β†’ `master`
* Add linter rules for naming convention

* Add rules for typeLike

* Add rules for destrctured variables

* Added changeset: Linter rules for naming convention

* Added changeset: Linter rules for naming convention
* initial update

* fix cloudpulse schemas

* fix incorrect `stackscript_data` schema

* add changesets

---------

Co-authored-by: Banks Nussman <banks@nussman.us>
…presentation (#11311)

* change: [UIE-8228] - DBaaS Resize GA: Enable Downsizing, update node presentation

* Added changeset: DBaaS Resize GA: Enable Downsizing (horizontal and vertical), enable 'Shared' tab, updated node presentation

* clean up unit test

* try to fix flake

* revert changes

---------

Co-authored-by: Banks Nussman <banks@nussman.us>
…r flow (#11322)

* Display new Cluster Type panel in create flow when LKE-E is enabled

* Disable LKE-E card without the account capability

* Require feature enablement to create LKE-E cluster

* Fix divider margin

* Change placement of tooltip

* Improve responsive styling

* Update useIsLkeEnterpriseEnabled hook test coverage

* Add test coverage to lke-create.spec.ts

* Add changeset

* Fix top margin of docs link

* Use the correct accountbeta query for the ClusterTypePanel

* clean up

---------

Co-authored-by: Hana Xu <hxu@akamai.com>
* KubeConfigDrawer

* KubeControlPlaneACLDrawer

* HistoryDrawer

* feedback @bnussman-akamai

* feedback @bnussman-akamai

* Fix passing data

* Update e2e lke update suite for better label

* fix maybe flaky DB test
* test: [M3-8546] - Add unit tests for DialogTitle component

* Added changeset: unit test cases for `DialogTitle` component

* Fix capitalization nitpicks in test file

* update changeset `Type` and `Description`
* Add new assertions for linode backup tests

* Update comments

* Added changeset: Add new assertions for linode backup tests

* update changeset comment
… load (#11356)

* Dont pass -1 as default

* Added changeset: LinodeCreate OS Panel fetches region with -1 on page load
Bumps [cross-spawn](https://github.com/moxystudio/node-cross-spawn) from 6.0.5 to 6.0.6.
- [Changelog](https://github.com/moxystudio/node-cross-spawn/blob/v6.0.6/CHANGELOG.md)
- [Commits](moxystudio/node-cross-spawn@v6.0.5...v6.0.6)

---
updated-dependencies:
- dependency-name: cross-spawn
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
)

* Move `FormControlLabel` to `ui` package

* Added changeset: Migrate `FormControlLabel` from `manager` to `ui` package

* Added changeset: Migrate `FormControlLabel` from `manager` to `ui` package
… dimensions (#11242)

* Customizing radio button sizes

* Added a sample

* Reverted sample

* Updated sizes for medium and small radio buttons

* Added changeset

* Wrapped radio icons in SvgIcon & added font sizes to themes/light.ts

* fixes

* linting fix
* bump recharts to ^2.14.1

* Added changeset: Bump recharts to ^2.14.1
…elerated plans (#11363)

* update link + add region availability banner

* Added changeset: Docs link and region availability notice for Accelerated compute plans

* update plan-selection tests due (new banner added)
…XR request (#11361)

* Change sanitized url path string to match UXR request

* Added changeset: Change Pendo sanitized URL path string
* Fix regression with AppBar

* feedback @mjac0bs
## Description πŸ“
LKE-Enterprise will not be available in all regions. It will be available in limited DCs initially, then expanded to all core DCs. We want to clearly communicate to users why they cannot select the same DCs for LKE-E as we offer for LKE. 

## Changes  πŸ”„
- Filter region select by the `Kubernetes Enterprise` capability if LKE-E is selected & show tooltip

## How to test πŸ§ͺ

### Prerequisites

(How to setup test environment)

- Ensure you have the LKE-E customer tag on your account (see Project Tracker)
- Point to the development environment

### Verification steps

(How to verify changes)

- [ ] Go to http://localhost:3000/kubernetes/create
- [ ] When LKE standard cluster type is selected, all regions with the Kubernetes capability are displayed.
- [ ] When the LKE enterprise cluster type is selected, regions are filtered display only those with the Kubernetes Enterprise capability.
  - [ ] A tooltip is displayed next to the region select that explains the reason for the limited list.
- [ ] When the lkeEnterprise feature is disabled, no region tooltip is visible.
- [ ] Confirm test coverage passes

```
yarn cy:run -s "cypress/e2e/core/kubernetes/lke-create.spec.ts"
```
* feat: [M3-8953] - OBJ Gen2 Updates

* Add changeset

* fix duplicate buckets landing page, may rework

* Update e2e tests

* Remove redundant beforeEach

* Small adjustment to request

* fix access keys e2e test

* Fix bucket-create-gen2.spec.ts failing tests @linode/frontend-sdet

* E2E review updates @cliu-akamai @AzureLatte

* Fix issue where properties content was overriding SSL for non-gen2 capability users @coliu-akamai

* Remove CORS from object drawers @bnussman-akamai

* Update packages/manager/.changeset/pr-11355-upcoming-features-1733237339445.md

Co-authored-by: Dajahi Wiley <114682940+dwiley-akamai@users.noreply.github.com>

* Update packages/manager/.changeset/pr-11355-upcoming-features-1733237339445.md

Co-authored-by: Dajahi Wiley <114682940+dwiley-akamai@users.noreply.github.com>

* Remove double copy icon @dwiley-akamai

* Add changesets

* Update packages/manager/src/features/ObjectStorage/AccessKeyLanding/AccessKeyTable/HostNameTableCell.tsx

Co-authored-by: Banks Nussman <115251059+bnussman-akamai@users.noreply.github.com>

* Pluralize the regions

* Fix E2E based in pluralize

* Fix unit tests

* Fix e2e from removing toggle

---------

Co-authored-by: Jaalah Ramos <jaalah.ramos@gmail.com>
Co-authored-by: Connie Liu <coliu@akamai.com>
Co-authored-by: Dajahi Wiley <114682940+dwiley-akamai@users.noreply.github.com>
Co-authored-by: Banks Nussman <115251059+bnussman-akamai@users.noreply.github.com>
…#11293)

* fix: [M3-8740] - Convert Object Storage size from GiB to GB in frontend

* Added changeset: Convert Object Storage bucket sizes from `GiB` to `GB` in the frontend

* refactor: [M3-8740] - Convert Object Storage size from GiB to GB in frontend

* Change comment description
abailly-akamai and others added 5 commits December 5, 2024 10:38
…x defaultDB for legacy instances (#11371)

* feat: [UIE-8269], [UIE-8290] - DBaaS: Access Controls text update, fix defaultDB for legacy instances

* feat: [UIE-8269], [UIE-8290] - refactoring

* feat: [UIE-8269], [UIE-8290] - fix e2e test

* Added changeset: DBaaS : Updated copy, placeholders, and button text in Access Controls, fixed default DB value for legacy instances
* update hookform resolvers and try to fix schema issues

* fix import

* fix acl regression

* revert linode create resolver change

---------

Co-authored-by: Banks Nussman <banks@nussman.us>
…gacy flow (#11300)

* fix: [M3-8912] - Create support ticket for bucket created through legacy flow

* Added changeset: Create support ticket for buckets created through legacy flow
* test: [M3-8758]- Flake  flaky test

* Added changeset: Fix `delete-volume.spec.ts` flaky test
@santoshp210-akamai santoshp210-akamai merged commit 291fadb into santoshp210-akamai:feature/list-alert-definition-landing-page Dec 6, 2024
8 checks passed
Copy link

github-actions bot commented Dec 6, 2024

Coverage Report: ❌
Base Coverage: 86.96%
Current Coverage: 86.85%

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.