Skip to content

chore: Remove unused container files#9928

Merged
jaalah-akamai merged 2 commits intolinode:developfrom
jaalah-akamai:login-cleanup
Nov 28, 2023
Merged

chore: Remove unused container files#9928
jaalah-akamai merged 2 commits intolinode:developfrom
jaalah-akamai:login-cleanup

Conversation

@jaalah-akamai
Copy link
Contributor

Description 📝

Doing some minor holiday cleanup

Changes 🔄

  • Deleted packages/manager/src/containers/events.container.ts
  • Deleted packages/manager/src/containers/withApplicationStore.container.tsx
  • Added minor comments to auth files

@jaalah-akamai jaalah-akamai self-assigned this Nov 22, 2023
@jaalah-akamai jaalah-akamai requested a review from a team as a code owner November 22, 2023 17:09
@jaalah-akamai jaalah-akamai requested review from bnussman-akamai, carrillo-erik and coliu-akamai and removed request for a team November 22, 2023 17:09
Copy link
Contributor

@coliu-akamai coliu-akamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the cleanup! 🧹
did a general sanity check of CM as well, things look good!

@coliu-akamai coliu-akamai added the Add'tl Approval Needed Waiting on another approval! label Nov 22, 2023
Copy link
Member

@bnussman-akamai bnussman-akamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉🦃

Might want to add a tech story changeset

@coliu-akamai coliu-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Nov 22, 2023
@jaalah-akamai jaalah-akamai merged commit e3619b0 into linode:develop Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Multiple approvals and ready to merge!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants