Skip to content

Removed v1 component split for event types component#5399

Merged
kodiakhq[bot] merged 8 commits into
mainfrom
chore/cleanup-v1-for-eventtype-components
Nov 10, 2022
Merged

Removed v1 component split for event types component#5399
kodiakhq[bot] merged 8 commits into
mainfrom
chore/cleanup-v1-for-eventtype-components

Conversation

@JeroenReumkens
Copy link
Copy Markdown
Contributor

What does this PR do?

Remove v1 split for eventtypes components in web app.

Environment: Staging(main branch)

Type of change

  • Chore (refactoring code, technical debt, workflow improvements)

How should this be tested?

  • There has been little change, mainly just moved components, but it might be good to quickly check the event types section to see if you don't see anything weird.

@vercel
Copy link
Copy Markdown

vercel Bot commented Nov 7, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cal ✅ Ready (Inspect) Visit Preview Nov 10, 2022 at 11:17AM (UTC)

@what-the-diff
Copy link
Copy Markdown

what-the-diff Bot commented Nov 7, 2022

  • Renamed all the components in apps/web/components/v2 to be just under apps/web
    • This was done because we are no longer using v1 and v2 of our UI library. We have merged them into one, so there is only a single version now.

@socket-security
Copy link
Copy Markdown

socket-security Bot commented Nov 8, 2022

Socket Security Pull Request Report

👍 No dependency changes detected in pull request

Pull request report summary
Issue Status
Install scripts ✅ 0 issues
Native code ✅ 0 issues
Bin script confusion ✅ 0 issues
Bin script shell injection ✅ 0 issues
Unresolved require ✅ 0 issues
Invalid package.json ✅ 0 issues
HTTP dependency ✅ 0 issues
Git dependency ✅ 0 issues
Potential typo squat ✅ 0 issues
Known Malware ✅ 0 issues
Telemetry ✅ 0 issues
Protestware/Troll package ✅ 0 issues

Powered by socket.dev

Copy link
Copy Markdown
Member

@CarinaWolli CarinaWolli left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@hariombalhara hariombalhara added ♻️ autoupdate tells kodiak to keep this branch up-to-date automerge labels Nov 10, 2022
@kodiakhq kodiakhq Bot merged commit 4176086 into main Nov 10, 2022
@kodiakhq kodiakhq Bot deleted the chore/cleanup-v1-for-eventtype-components branch November 10, 2022 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge ♻️ autoupdate tells kodiak to keep this branch up-to-date

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants