Skip to content

fix: showing default apps by Cal like Cal-Video in installed apps#10611

Merged
hariombalhara merged 3 commits intocalcom:mainfrom
satya-nishanth:CAL-2319
Aug 10, 2023
Merged

fix: showing default apps by Cal like Cal-Video in installed apps#10611
hariombalhara merged 3 commits intocalcom:mainfrom
satya-nishanth:CAL-2319

Conversation

@satya-nishanth
Copy link
Copy Markdown
Contributor

@satya-nishanth satya-nishanth commented Aug 5, 2023

What does this PR do?

This PR fixes the problem where default apps like cal-video are getting filtering out from installed apps

Fixes #10595

Earlier without this fix

Screenshot 2023-08-05 at 10 29 29 PM

After this fix
Screenshot 2023-08-05 at 10 29 19 PM

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How should this be tested?

go to /apps/installed/conferencing now you can see cal-video

Mandatory Tasks

  • Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

Checklist

  • I haven't checked if new and existing unit tests pass locally with my changes

@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 5, 2023

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

Name Status Preview Comments Updated (UTC)
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 9, 2023 1:28pm

@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 5, 2023

Someone is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added the app-store area: app store, apps, calendar integrations, google calendar, outlook, lark, apple calendar label Aug 5, 2023
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Aug 5, 2023

Thank you for following the naming conventions! 🙏

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Aug 5, 2023

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@PeerRich PeerRich requested a review from joeauyeung August 7, 2023 15:22
@PeerRich PeerRich added the ⚡ Quick Wins A collection of quick wins/quick fixes that are less than 30 minutes of work label Aug 7, 2023
Comment thread packages/lib/apps/getEnabledApps.ts Outdated
@satya-nishanth
Copy link
Copy Markdown
Contributor Author

please review @joeauyeung and thanks for the feedback

Copy link
Copy Markdown
Member

@hariombalhara hariombalhara left a comment

Choose a reason for hiding this comment

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

Let's improve the perf as a followup. Thanks @satya-nishanth 🙏

@hariombalhara hariombalhara dismissed joeauyeung’s stale review August 10, 2023 10:33

Performance to be improved in a followup. Bugfix first.

@hariombalhara hariombalhara merged commit 22b4bcd into calcom:main Aug 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

app-store area: app store, apps, calendar integrations, google calendar, outlook, lark, apple calendar 🐛 bug Something isn't working ⚡ Quick Wins A collection of quick wins/quick fixes that are less than 30 minutes of work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CAL-2319] When you delete CalVideo from the installed apps page, you can't install it back

4 participants