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

M3 9066 remove cypress deprecated helperts functions #11496

Conversation

dmcintyr-akamai
Copy link
Contributor

@dmcintyr-akamai dmcintyr-akamai commented Jan 9, 2025

Description 📝

Remove deprecated helper functions.

Changes 🔄

List any change(s) relevant to the reviewer.

-Removed deprecated helper functions from helper.ts
-Replaced these helper functions with the cypress methods wrapped by the deprecated functions
-My branch still has 20 tests that are failing, but none of the errors are in files that I modified in this branch.
-All code changes are in cypress tests in packages/manager/cypress, no changes to production code.

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 🧪

yarn cy:run

Prerequisites

none
(How to setup test environment)

  • ...
  • ...

Reproduction steps

(How to reproduce the issue, if applicable)

  • ...
  • ...

Verification steps

All tests in modified files should pass.
(How to verify changes)

  • No additional test failures beyond what are already occurring in develop branch
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


  • [x ] 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

@dmcintyr-akamai dmcintyr-akamai requested review from a team as code owners January 9, 2025 14:27
@dmcintyr-akamai dmcintyr-akamai requested review from jdamore-linode, bnussman-akamai and pmakode-akamai and removed request for a team January 9, 2025 14:27
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 23 failing tests on test run #1 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
23 Failing443 Passing2 Skipped77m 17s

Details

Failing Tests
SpecTest
clone-linode.spec.tsclone linode » can clone a Linode from Linode details page
service-transfer.spec.tsAccount service transfers » can initiate and cancel a service transfer
search-images.spec.tsSearch Images » creates two images and make sure they show up in the table and are searchable
update-firewall.spec.tsupdate firewall » updates a firewall's linodes and rules
update-firewall.spec.tsupdate firewall » updates a firewall's status
update-firewall.spec.tsupdate firewall » updates a firewall's label
create-volume.spec.tsvolume create flow » creates an attached volume
create-volume.spec.tsvolume create flow » displays a warning notice on Volume Create page re: rebooting for client library updates under the appropriate conditions
create-volume.spec.tsvolume create flow » displays a warning notice re: rebooting for client library updates under the appropriate conditions in Create/Attach Volume drawer
create-volume.spec.tsvolume create flow » creates a volume from an existing Linode
create-firewall.spec.tscreate firewall » creates a firewall assigned to a linode
migrate-linode-with-firewall.spec.tsMigrate Linode With Firewall » migrates linode with firewall - real data
smoke-delete-linode.spec.tsdelete linode » deletes linode from linode details page
smoke-delete-linode.spec.tsdelete linode » deletes linode from setting tab in linode details page
smoke-delete-linode.spec.tsdelete linode » deletes linode from linode landing page
smoke-delete-linode.spec.tsdelete linode » deleting multiple linodes with action menu
create-stackscripts.spec.tsCreate stackscripts » creates a StackScript with Any/All target image
rebuild-linode.spec.tsrebuild linode » rebuilds a linode from Image
rebuild-linode.spec.tsrebuild linode » rebuilds a linode from Community StackScript
attach-volume.spec.tsvolume attach and detach flows » attaches a volume to a Linode
backup-linode.spec.tslinode backups » can enable backups
backup-linode.spec.tslinode backups » can capture a manual snapshot
rescue-linode.spec.tsRescue Linodes » Can reboot a Linode into rescue mode

Troubleshooting

Use this command to re-run the failing tests:

yarn cy:run -s "cypress/e2e/core/linodes/clone-linode.spec.ts,cypress/e2e/core/account/service-transfer.spec.ts,cypress/e2e/core/images/search-images.spec.ts,cypress/e2e/core/firewalls/update-firewall.spec.ts,cypress/e2e/core/firewalls/update-firewall.spec.ts,cypress/e2e/core/firewalls/update-firewall.spec.ts,cypress/e2e/core/volumes/create-volume.spec.ts,cypress/e2e/core/volumes/create-volume.spec.ts,cypress/e2e/core/volumes/create-volume.spec.ts,cypress/e2e/core/volumes/create-volume.spec.ts,cypress/e2e/core/firewalls/create-firewall.spec.ts,cypress/e2e/core/firewalls/migrate-linode-with-firewall.spec.ts,cypress/e2e/core/linodes/smoke-delete-linode.spec.ts,cypress/e2e/core/linodes/smoke-delete-linode.spec.ts,cypress/e2e/core/linodes/smoke-delete-linode.spec.ts,cypress/e2e/core/linodes/smoke-delete-linode.spec.ts,cypress/e2e/core/stackscripts/create-stackscripts.spec.ts,cypress/e2e/core/linodes/rebuild-linode.spec.ts,cypress/e2e/core/linodes/rebuild-linode.spec.ts,cypress/e2e/core/volumes/attach-volume.spec.ts,cypress/e2e/core/linodes/backup-linode.spec.ts,cypress/e2e/core/linodes/backup-linode.spec.ts,cypress/e2e/core/linodes/rescue-linode.spec.ts"

jaalah-akamai and others added 18 commits January 9, 2025 11:39
* docs: [M3-9010] - UseEffect Best Practices

* Update docs/development-guide/13-coding-standards.md

Co-authored-by: Dajahi Wiley <[email protected]>

* Added changeset: Update our docs regarding useEffect best practices

---------

Co-authored-by: Jaalah Ramos <[email protected]>
Co-authored-by: Dajahi Wiley <[email protected]>
…or tokens pt4 (linode#11345)

* Replace one-off hardcoded color values with color tokens pt4

* Replace `#32363c` `#888f91` `linode#999` `#fbfbfb` `#3f8a4e` `#4FAD62` `#72BD81` `#ADD8E6` `#eaeaf0` `#5c6470` with color tokens

* Replace `#9DA4A6` `#c4c4c4` `#D6D7D9` with color tokens

* Added changeset: Replace one-off hardcoded color values with color tokens pt4

---------

Co-authored-by: Purvesh Makode <[email protected]>
…y selecting a VPC as the primary interface (linode#11424)

* set initial primaryInterfaceIndex to 0

* extra space oops

* some type cleanup

* Added changeset: Linode Edit Config warning  message when initially selecting a VPC as the primary interface
…ode#11391)

* Initial commit - save progress

* styling

* Switching gears a little

* save progress

* save progress

* JSDOCs

* Better creatable logic

* clenup logic

* save progress

* Decent cleanup

* save progress

* test progress

* Unit test

* component test

* replace MonitorDrawer selects

* cleanup

* Changesets

* Type tightening

* more Type tightening

* cleanup

* feedback @mjac0bs
…de#11392)

* upcoming: [DI-22184] - Added Metric, MetricCriteria components with Unit Tests and minor changes to align with changes in API spec

* upcoming: [DI-22184] - Internal Review Changes: Assertion checks in Unit tests, added a type for the Form, renamed prop

* upcoming: [DI-22184] - Removing few initial values used for internal testing

* upcoming: [DI-22184] - Fixing previoius accidental commit

* upcoming: [DI-22184] - Made the Clear icon a separate Component with Unit Test

* upcoming: [DI-22184] - Added changesets

* upcoming: [DI-22184] - Review changes: replaced watch with useWatch, removed non-null assertion and using default values while filtering form values, modified code to include just side effects in the useEffect in MetricCriteria component

* upcoming: [DI-22184] - Removed braces where values where plain strings

* upcoming: [DI-22184] - Renamed the variable for error and loading state for better clarity

* upcoming: [DI-22184] - Display maxScrapeInterval to pass the typecheck-manager test
…r LKE-E (linode#11415)

* Make a hook because this is getting repetitive

* Determine enterprise or standard versions with hook

* Update versions type for util - need to test

* Improve hook and use in cluster create flow

* Move hook to kubeUtils

* Add test coverage and factories

* Clean up kubeUtils.test.ts

* Add kubeUtils helper function test coverage for LKE-E

* Update Cypress test coverage for enterprise version upgrades

* Clean up

* Added changeset: Update Kubernetes version upgrade components for LKE-E

* Fix test failure by mocking the LKE-E capability needed

* Update TODO

* Address PR feedback: increment ids in tiered version factories
* surface tags for node pools

* wip e2e

* fix e2e

* Added changeset: Surface Node Pool Tags

* fix unit test

* Added changeset: tags to KubeNodePoolResponse and CreateNodePoolData

* clean up

* address feedback

* clean up

* fix tags cutting off in mobile

* spacing

* use md breakpoint instead of sm

* styling
* treat scripts as a workspace

* update package script

* update more scripts

* update more scripts

* update path resolution in `junit-summary/index.ts`

* make pipeline scripts work correctly

* fix broken `package-versions` script

---------

Co-authored-by: Banks Nussman <[email protected]>
…ode#11412)

* fix : [M3-7528] - Account Cancellation Survey Button Color Issues

* Add changeset
* Update Pendo URL with CNAME

* Update Analytics docs

* Added changeset: Update Pendo URL with CNAME and update Analytics developer docs

* Update docs/tooling/analytics.md

Co-authored-by: Connie Liu <[email protected]>

* Update docs/tooling/analytics.md

Co-authored-by: Connie Liu <[email protected]>

---------

Co-authored-by: Connie Liu <[email protected]>
…1419)

* get v4beta/lke/types when lke-e is selected

* rename cluster type heading to cluster tier

* add lke enterprise pricing to checkout summary

* move HA and enterprise pricing to top of checkout summary

* styling clean up and rename file

* fix disabled create button

* remove HA choice wording from checkout message for lke-e

* Added changeset: Display LKE-E pricing in checkout bar

* Added changeset: Added `getKubernetesTypesBeta` function

* update e2e

* Finish out the Cypress test to create an LKE-E cluster

* Update packages/api-v4/.changeset/pr-11419-upcoming-features-1734115734709.md

Co-authored-by: Dajahi Wiley <[email protected]>

* Update packages/manager/src/features/Kubernetes/KubeCheckoutBar/KubeCheckoutBar.tsx

Co-authored-by: Dajahi Wiley <[email protected]>

* Lint and address feedback: styling clean up

* Fix CTA button disabled without enterprise tier HA default (@harsh-akamai)

* Fix prop issue from merge

* Update HA state for LKE tier too after e976a38

* Use undefined not false, to match the HA radios

---------

Co-authored-by: mjac0bs <[email protected]>
Co-authored-by: Mariah Jacobs <[email protected]>
Co-authored-by: Dajahi Wiley <[email protected]>
…1333)

* initial idea

* improve comment

* fix spelling

* fix spelling

* use `to` rather than `link`

* use `to` rather than `link`

* fix comment

* use the new prop in Cloud Manager

* impliment some changes from Jaalah's component

* add changeset

* thanks @abailly-akamai 🙏

* clean up a bit

* final clean up

---------

Co-authored-by: Banks Nussman <[email protected]>
…e#11388)

* change: [DI-22125] - Moved time range select outside the paper in contextual view

* change: [DI-22125] - Moved duplicate code into utils

* change: [DI-22125] - Added test cases

* change: [DI-22125] - Added default label to contextual view

* Added changeset

* change: [DI-22125] - Updated mock data

* upcoming: [DI-22125] - Updated logic to hide label in contextual view

* upcoming: [DI-22125] - Removed hide label from global filters

* change: [DI-22390] - Added disabled dashboard select in contextual view

* change: [DI-22390] - Updated test cases

* change: [DI-22125] - Updated styling
…yword (linode#11385)

* fix: [M3-8905] - Change document title from address to appropriate keyword

* Added changeset: Document title from URL to appropriate keyword

* Make Document title more descriptive

* resolve small regression

* Change some more document title from address to appropriate keyword
…nding page (linode#11335)

* test: [M3-8608] - Cypress test for restricted user Image non-empty landing page

* Move function outside describe block and use `closest()` locator to locate tables

* Added changeset: Cypress test for restricted user Image non-Empty landing page

* Move `checkActionMenu` function above `describe` block
* feat: [M3-8177] - Add MultipleIPInput Story

* Added changeset: MultipleIPInput Story in Storybook

* Add functionality to `HelperText` and `Placeholder`

* Add functionality to `Placeholder`
* save progress

* SURGERY

* Implement basic routing

* Wrap up routing

* fix unit

* routes test update

* better handling in test

* fix smoke test

* Added changeset: Refactor Domains Feature Routing

* MSW fix

* edit changeset

* mock path consistency

* feedback @dwiley-akamai

* feedback @mjac0bs

* fix e2e
mjac0bs and others added 26 commits January 9, 2025 11:40
…linode#11442)

* Create component for rendering cluster chips

* Add test coverage for new component

* Replace current chip with ClusterChips component

* Fix spacing in column specs due to long LKE-E version

* Remove Kube dashboard button for LKE-E clusters

* Fix the fix for specs spacing

* Add changeset

* Fix a conditional before I forget

* Address feedback - clean up flag mocks in test

* Update packages/manager/src/features/Kubernetes/ClusterList/ClusterChips.test.tsx

Co-authored-by: Hana Xu <[email protected]>

* Update packages/manager/src/features/Kubernetes/KubernetesClusterDetail/KubeClusterSpecs.tsx

Co-authored-by: Hana Xu <[email protected]>

---------

Co-authored-by: Hana Xu <[email protected]>
)

* Fix NodeBalancersCreate flow console errors

* Added changeset: `RegionHelperText` causing console errors

* Props fixes
* update `react-vnc` to `2.0.2`

* add changeset

---------

Co-authored-by: Banks Nussman <[email protected]>
…rt details page in Cloud Pulse. (linode#11466)

* upcoming: [DI-22596] - Overview changes

* upcoming: [DI-22131] - CSS and factory updates for alert detail overview

* upcoming: [DI-22131] - Code clean up and refactoring

* upcoming: [DI-22131] - Type updates and es lint fixes

* upcoming: [DI-22131] - Add more UT's

* upcoming: [DI-22131] - Add changeset

* upcoming: [DI-22131] - Code reusability

* upcoming: [DI-22131] - Code refactoring, comments update

* upcoming: [DI-22131] - Code refactoring, todo comments update

* upcoming: [DI-22131] - Code refactoring

* upcoming: [DI-22131] - Code refactoring

* upcoming: [DI-22131] - Code refactoring

* upcoming: [DI-22131] - Edit Changeset

* upcoming: [DI-22131] - Typography font updates

* upcoming: [DI-22131] - Util function test optimisation

* upcoming: [DI-22131] - Code clean up and refactoring

* upcoming: [DI-22131] - Code clean up and refactoring

* upcoming: [DI-22131] - Code clean up and refactoring

* upcoming: [DI-22131] - Use imports from linode UI

* upcoming: [DI-22131] - Update UT's

* upcoming: [DI-22131] - Comment updates

* upcoming: [DI-22131] - Comment updates

* upcoming: [DI-22131] - Rename service type to service type list

* upcoming: [DI-22131] - Use shared mock data

* upcoming: [DI-22131] - Changed text and background color

---------

Co-authored-by: vmangalr <[email protected]>
…11457)

* upcoming: [DI-20935] - Add support for tags filter

- Added tag filter selection component
- Made tag filter as an optional dependency
- Passed tag as an x-filter to filter resources
- Included exhaustive unit test cases
- Updated mocks for handling tags

* upcoming: [DI-20935] - updated mocks

* upcoming: [DI-20935] - updated UTs

* upcoming: [DI-20935] - small fix

* upcoming: [DI-20935] - small fix

* upcoming: [DI-20935] - add required field for custom select, update UT

* upcoming: [DI-20935] - PR comments

* upcoming: [DI-20935] - PR comments

* upcoming: [DI-20935] - small fix

* upcoming: [DI-20935] - test case fix

* upcoming: [DI-20935] - Add optional subtitle on tags, remove required from other filters

* upcoming: [DI-20935] - PR comments

* upcoming: [DI-20935] - small fix

* upcoming: [DI-20935] - small fix

* upcoming: [DI-20935] - small fix

* upcoming: [DI-20935] - PR Comments

* upcoming: [DI-20935] - prop drill optional

* upcoming: [DI-20935] - linting fix

* upcoming: [DI-20935] - mock enhancement

* upcoming: [DI-20935] - small preferences fix

* upcoming: [DI-20935] - Add changeset

* upcoming: [DI-20935] - resolve pr comments
…ode#11405)

* add initial udp support

* clean up and improve

* more progress

* small ui improvement

* more ui fixes

* improve types by making `configIdx` required

* add changeset

* add some basic testing

* reimpliment changes I accidentally deleted

* more progress

* clean up and improve a bit

* clean up

* Apply suggestions from code review

Co-authored-by: Mariah Jacobs <[email protected]>

* clean up constants

* clean up constants a bit more

* fix test that I broke

* work on addressing feedback

---------

Co-authored-by: Banks Nussman <[email protected]>
Co-authored-by: Mariah Jacobs <[email protected]>
… monthly pricing (linode#11475)

* Update price on cluster details page to include enterprise

* Update test coverage to confirm LKE-E creation details

* Added changeset: Update LKE cluster details kube specs for LKE-E monthly pricing
)

## Description 📝
Fix region validation when a user is switching from LKE to LKE-E tier in the Kubernetes create flow

## How to test 🧪

### Prerequisites

(How to setup test environment)

- Ensure you have the customer tags for LKE-E (check project tracker)

### Reproduction steps

(How to reproduce the issue, if applicable)

- [ ] Go to another branch or dev environment and navigate to the Kubernetes create page
- [ ] Enter a cluster label, select LKE as the tier, select a non LKE-E supported region (i.e. Atlanta, GA), select a HA control plane option, add a plan
- [ ] Change the selected tier to LKE-E. Notice the region select is cleared but the Create Cluster button is enabled. Clicking Create Cluster shows an error text in the Region field

### Verification steps

(How to verify changes)

- [ ] Checkout this branch or PR preview link and navigate to the Kubernetes create page
- [ ] Enter a cluster label, select LKE as the tier, select a non LKE-E supported region (i.e. Atlanta, GA), select a HA control plane option, add a plan
- [ ] Change the selected tier to LKE-E. The region select should be cleared and the Create Cluster button is disabled
…inode#11440)

* test: [M3-8661] - Cypress test to validate CAA record are editable

* Added changeset: Cypress test to validate CAA records are editable

* Splitting the test cases for valid and invalid records

* Move`createCaaRecord` and `verifyRecordInTable` function calls in `beforeEach` block

* Add cleanup, use @example.com for emails, and replace `cy.get()` for stability.

* Update Changeset type
* autocomplete tests

* eslint

* cleanup tests

* Added changeset: Add cypress component tests for Autocomplete

* address feedback @jdamore-linode

* Update packages/manager/cypress/component/poc/autocomplete.spec.tsx

Co-authored-by: jdamore-linode <[email protected]>

* create components package within ...component... package

---------

Co-authored-by: jdamore-linode <[email protected]>
…age (linode#11478)

## Description 📝
Add coverage for checking that there is no upgrade chip when there is no new kube version available and move kube version landing page e2e tests from `lke-update.spec.ts` to `lke-landing-page.spec.ts`

## How to test 🧪

```
yarn cy:run -s "cypress/e2e/core/kubernetes/lke-landing-page.spec.ts"
```

---------

Co-authored-by: Mariah Jacobs <[email protected]>
* Add test for LKE cluster rename flow

* Added changeset: Add test for LKE cluster rename flow

* Fix typo in test name

---------

Authored-by: Azure Shi <[email protected]>
Co-authored-by: Mariah Jacobs <[email protected]>
…1456)

* refactor: [M3-6910] - Replace instances in : Object Storage

* Added changeset: Replace Select with Autocomplete component in Object Storage

* Remove unnecessary `vi.mocks`
…ions selected (linode#11453)

* fix: Enable `Create Token` button if all visible perms selected

* Added changeset: `Create Token` button becomes disabled when all permissions are selected individually (without using 'select all') and child-account is hidden

* Update the in-line doc
* deps: [M3-8782] - Patch cookie version

* Added changeset: Patch `cookie` version as resolution for dependabot

* Add lockfile

---------

Co-authored-by: Jaalah Ramos <[email protected]>
* fix: [M3-9060] - Object Storage  sorting

* Add changeset

---------

Co-authored-by: Jaalah Ramos <[email protected]>
…ing Cloud images (linode#11360)

* Allow Docker registry to be overridden via `IMAGE_REGISTRY` arg

* Add changeset
…11476)

* Add maskable text to Managed Contacts

* Update the docs link URL to a more helpful doc

* Make component for reusable copy

* Use component in Billing Contact Info and SSH Access Pub Key

* Make MaskableText icon styling more flexible; add to Longview command

* Add masking to Longview API Key

* Clean up styles

* Mask IP in Longview

* Mask IP Address on SSH Access Managed tab

* Fix margin for API Key

* Fix spacing for Longview API key

* Mask user in Managed and Longview

* Added changeset: Visibility of sensitive data in Managed and Longview with Mask Sensitive Data setting enabled
…omponents (linode#11445)

* upcoming: [DI-22558] - Added Trigger condition and Dimension Filter changes

* removed unnecessary console log statement

* upcoming: [DI-22558] - Importing right interface for Trigger condition for error validation and minor changes

* upcoming: [DI-22588] - Added the Unit Tests for the components

* upcoming: [DI-22588] - Added changesets and few changes to the messages in validation schema

* upcoming: [DI-22588] - Review comments and made some minor UX changes to the component

* Updated stylings

* upcoming: [DI-22588] - UX changes

* upcoming: [DI-22588] - Fixed the validation error resolution for Metric Data Field and Dimension Filter Data Field

* upcoming: [DI-22588] - Review changes

* upcoming: [DI-22588] - Moved mockData to individual test files instead of exporting to fix the failing tests

* upcoming: [DI-22558] - Typescript issue fixes

* Update constants.ts

---------

Co-authored-by: nikhagra-akamai <[email protected]>
Co-authored-by: vmangalr <[email protected]>
Co-authored-by: venkatmano-akamai <[email protected]>
…es by default (linode#11486)

* Use Debian 12 for Linode factories, replace Debian 10 with 12 in tests

* Added changeset: Fix Cypress test failures stemming from Debian 10 Image deprecation

* Fix StackScript test by resizing disk to be slightly larger for Debian 12
* add MSW crud domains

* Added changeset: Add MSW crud domains

* feedback @jaalah-akamai

* feebback @pmakode-akamai
…inode#11485)

## Description 📝
Due to the way that existing code was written, we were only returning Kubernetes API endpoints for clusters if the endpoint matched port 443 (e.g. `/linodelke\.net:443`). However, LKE-E endpoints use the `6443` port so nothing was returning

This PR adds a check to render the first available endpoint if port 443 is not available

## Changes  🔄

List any change(s) relevant to the reviewer.

- Render the first available endpoint if endpoint with port 443 is not available
- Added some unit tests

## How to test 🧪

### Prerequisites

(How to setup test environment)

- Ensure you have LKE-E customer tags (check project tracker)
- Have an LKE-E cluster created

### Reproduction steps

(How to reproduce the issue, if applicable)

- [ ] On another branch (e.g. `develop`), go to the details page of an LKE-E cluster
- [ ] The Kubernetes API endpoint displays `Your endpoint will be displayed here once it is available.`

### Verification steps

(How to verify changes)

- [ ] Checkout this PR or use the preview link, go to the details page of an LKE-E cluster
- [ ] The Kubernetes API endpoint displays the first endpoint available (should be something like `https://lke287706.us-ord.enterprise.linodelke.net:6443`)
…1491)

* change: [M3-9074] - Remove properties tab from Gen2 buckets

* Added changeset: Temporarily remove Properties tab from Gen2 buckets

---------

Co-authored-by: Jaalah Ramos <[email protected]>
* upcoming: [M3-9008] Improve Close Account UI

* Add changesets

* Update e2e test to cover parent/child

* More cleanup

* Change warning to error and update variants/colors

* Enforce type-to-confirm for close account

* Small revert

* Revision

* Text updates to title, warning, comment blurb

* Expand comments section full width - allow this prop to be set

* Fix unit tests due to usePreference updates - remove loading variant

* Update packages/manager/cypress/e2e/core/account/account-cancellation.spec.ts

Co-authored-by: Connie Liu <[email protected]>

* Update packages/manager/cypress/e2e/core/account/account-cancellation.spec.ts

Co-authored-by: Connie Liu <[email protected]>

* Update packages/manager/cypress/e2e/core/account/account-cancellation.spec.ts

Co-authored-by: Connie Liu <[email protected]>

* Update state variable name @coliu-akamai

* Allow expand to be passed for deletionDialogs

* Expand all delete type-to-confirms @coliu-akamai

* more deletion dialogs

* Few more dialogs

---------

Co-authored-by: Jaalah Ramos <[email protected]>
Co-authored-by: Connie Liu <[email protected]>
…nent. (linode#11395)

* unit test coverage for HostNameTableCell

* Revert "unit test coverage for HostNameTableCell"

This reverts commit b274baf.

* chore: [M3-8662] - Update Github Actions actions (linode#11009)

* update actions

* add changeset

---------

Co-authored-by: Banks Nussman <[email protected]>

* Basci date picker component

* Test coverage for date picker component

* DatePicker Stories

* Custom DateTimePicker component

* Reusable TimeZone Select Component

* Create custom DateTimeRangePicker component

* Storybook for DateTimePicker

* Fix tests and remove console warnings

* changeset

* Update packages/manager/src/components/DatePicker/DateTimeRangePicker.tsx

Co-authored-by: Connie Liu <[email protected]>

* Adjust styles for DatePicker

* Adjust styles for DateTimePicker

* update imports

* Render time and timezone conditionally in DateTimePicker component

* Move DatePicker to UI package

* Add DatePicker dependencies

* Code cleanup

* PR feedback

* code cleanup

* Move DatePicker back to src/components

* Reverting changes

* Code cleanup

* Adjust broken tests

* Update TimeZoneSelect.tsx

* Code cleanup

* Add validation for start date agains end date.

* Adjust styles for TimePicker component.

* Add the functionality to support Date Presets

* Update presets functionality and add test coverage.

* Added changeset: Add Date Presets Functionality to Date Picker component

* Persist the preset value

* Show the start date and end date fields only when custom is selected

* Add calendar icon to DateTimePicker component

* code cleanup and adjust tests

* Update packages/manager/src/components/DatePicker/DateTimeRangePicker.tsx

Co-authored-by: Nikhil Agrawal <[email protected]>

* update components

* Organize and additional prop support to DateTimeRangePicker component

* Code cleanup

* PR feedback - @coliu-akamai

* Move styles to theme level

* Revert "Move styles to theme level"

This reverts commit 15d9134.

* code cleanup

---------

Co-authored-by: Banks Nussman <[email protected]>
Co-authored-by: Banks Nussman <[email protected]>
Co-authored-by: Connie Liu <[email protected]>
Co-authored-by: Nikhil Agrawal <[email protected]>
@dmcintyr-akamai
Copy link
Contributor Author

Closing this pr bc i couldn't properly rebase my branch onto the updated develop branch, so created new branch and cherrypicked my commits into the new branch

@dmcintyr-akamai dmcintyr-akamai deleted the M3-9066-remove-cypress-deprecated-helperts-functions branch January 9, 2025 17:18
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.