Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@rjackson is attempting to deploy a commit to the cal Team on Vercel. A member of the Team first needs to authorize it. |
📦 Next.js Bundle Analysis for @calcom/webThis analysis was generated by the Next.js Bundle Analysis action. 🤖 Eight Pages Changed SizeThe following pages changed size from the code in this PR compared to its base branch:
DetailsOnly the gzipped size is provided here based on an expert tip. First Load is the size of the global bundle plus the bundle for the individual page. If a user were to show up to your website and land on a given page, the first load size represents the amount of javascript that user would need to download. If Any third party scripts you have added directly to your app using the The "Budget %" column shows what percentage of your performance budget the First Load total takes up. For example, if your budget was 100kb, and a given page's first load size was 10kb, it would be 10% of your budget. You can also see how much this has increased or decreased compared to the base branch of your PR. If this percentage has increased by 20% or more, there will be a red status indicator applied, indicating that special attention should be given to this. If you see "+/- <0.01%" it means that there was a change in bundle size, but it is a trivial enough amount that it can be ignored. |
|
amazing PR + description! thank you so much. super quick too |
|
@hariombalhara can you look into this:
|
|
@rjackson can you remove Web3? Huddle can be moved into conferencing |
|
i saw you went above and beyond, adding another bounty! |
|
/tip 100 |
|
🎉🎈 @rjackson has been awarded $100! 🎈🎊 |
zomars
left a comment
There was a problem hiding this comment.
Codewise LGTM. Delegating tests to QA 🙏🏽
|
Thank you for following the naming conventions! 🙏 |
That's so cool @rjackson. Thank you 🙏 |
|
PR updated with an environmental variable, The syncAppMeta endpoint only executes if I've also swapped over from Dry run output @calcom/web:dev: 13:30:08.936 INFO [api/cron/syncAppMeta] (dry-run) 🧐 Checking DB apps are in-sync with app metadata
@calcom/web:dev: 13:30:08.946 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app daily-video with categories
@calcom/web:dev: 13:30:08.946 INFO [api/cron/syncAppMeta] (dry-run) ✅ App apple-calendar is up-to-date and correct
@calcom/web:dev: 13:30:08.946 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app routing-forms with categories
@calcom/web:dev: 13:30:08.946 INFO [api/cron/syncAppMeta] (dry-run) ✅ App caldav-calendar is up-to-date and correct
@calcom/web:dev: 13:30:08.947 INFO [api/cron/syncAppMeta] (dry-run) ✅ App plausible is up-to-date and correct
@calcom/web:dev: 13:30:08.947 INFO [api/cron/syncAppMeta] (dry-run) ✅ App google-calendar is up-to-date and correct
@calcom/web:dev: 13:30:08.947 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app whereby with categories
@calcom/web:dev: 13:30:08.947 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app google-meet with categories
@calcom/web:dev: 13:30:08.947 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app zohocrm with categories
@calcom/web:dev: 13:30:08.947 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app jitsi with categories
@calcom/web:dev: 13:30:08.947 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app around with categories
@calcom/web:dev: 13:30:08.948 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app wipe-my-cal with categories
@calcom/web:dev: 13:30:08.948 INFO [api/cron/syncAppMeta] (dry-run) ✅ App wordpress is up-to-date and correct
@calcom/web:dev: 13:30:08.948 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app huddle01 with categories
@calcom/web:dev: 13:30:08.948 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app riverside with categories
@calcom/web:dev: 13:30:08.948 INFO [api/cron/syncAppMeta] (dry-run) ✅ App ga4 is up-to-date and correct
@calcom/web:dev: 13:30:08.948 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app typeform with categories
@calcom/web:dev: 13:30:08.948 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app webex with categories
@calcom/web:dev: 13:30:08.948 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app ping with categories
@calcom/web:dev: 13:30:08.948 INFO [api/cron/syncAppMeta] (dry-run) ✅ App pipedream is up-to-date and correct
@calcom/web:dev: 13:30:08.949 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app campfire with categories
@calcom/web:dev: 13:30:08.949 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app raycast with categories
@calcom/web:dev: 13:30:08.949 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app sirius_video with categories
@calcom/web:dev: 13:30:08.949 INFO [api/cron/syncAppMeta] (dry-run) ✅ App n8n is up-to-date and correct
@calcom/web:dev: 13:30:08.949 INFO [api/cron/syncAppMeta] (dry-run) ✅ App cron is up-to-date and correct
@calcom/web:dev: 13:30:08.949 INFO [api/cron/syncAppMeta] (dry-run) ✅ App exchange is up-to-date and correct
@calcom/web:dev: 13:30:08.949 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app sendgrid with categories
@calcom/web:dev: 13:30:08.949 INFO [api/cron/syncAppMeta] (dry-run) ✅ App qr_code is up-to-date and correct
@calcom/web:dev: 13:30:08.950 INFO [api/cron/syncAppMeta] (dry-run) ✅ App weather_in_your_calendar is up-to-date and correct
@calcom/web:dev: 13:30:08.950 INFO [api/cron/syncAppMeta] (dry-run) ✅ App gtm is up-to-date and correct
@calcom/web:dev: 13:30:08.950 INFO [api/cron/syncAppMeta] (dry-run) ✅ App fathom is up-to-date and correct
@calcom/web:dev: 13:30:08.950 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app closecom with categories
@calcom/web:dev: 13:30:08.950 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app whatsapp with categories
@calcom/web:dev: 13:30:08.950 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app discord with categories
@calcom/web:dev: 13:30:08.950 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app telegram with categories
@calcom/web:dev: 13:30:08.951 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app signal with categories
@calcom/web:dev: 13:30:08.951 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app sylapsvideo with categories
@calcom/web:dev: 13:30:08.951 INFO [api/cron/syncAppMeta] (dry-run) ✅ App vimcal is up-to-date and correct
@calcom/web:dev: 13:30:08.951 INFO [api/cron/syncAppMeta] (dry-run) ✅ App amie is up-to-date and correct
@calcom/web:dev: 13:30:08.951 INFO [api/cron/syncAppMeta] (dry-run) 🔨 Updating app facetime with categoriesLive run output: @calcom/web:dev: 13:30:47.140 INFO [api/cron/syncAppMeta] 🧐 Checking DB apps are in-sync with app metadata
@calcom/web:dev: 13:30:47.210 INFO [api/cron/syncAppMeta] 🔨 Updating app daily-video with categories
@calcom/web:dev: 13:30:47.220 INFO [api/cron/syncAppMeta] ✅ App apple-calendar is up-to-date and correct
@calcom/web:dev: 13:30:47.220 INFO [api/cron/syncAppMeta] 🔨 Updating app routing-forms with categories
@calcom/web:dev: 13:30:47.224 INFO [api/cron/syncAppMeta] ✅ App caldav-calendar is up-to-date and correct
@calcom/web:dev: 13:30:47.225 INFO [api/cron/syncAppMeta] ✅ App plausible is up-to-date and correct
@calcom/web:dev: 13:30:47.225 INFO [api/cron/syncAppMeta] ✅ App google-calendar is up-to-date and correct
@calcom/web:dev: 13:30:47.225 INFO [api/cron/syncAppMeta] 🔨 Updating app whereby with categories
@calcom/web:dev: 13:30:47.230 INFO [api/cron/syncAppMeta] 🔨 Updating app google-meet with categories
@calcom/web:dev: 13:30:47.234 INFO [api/cron/syncAppMeta] 🔨 Updating app zohocrm with categories
@calcom/web:dev: 13:30:47.237 INFO [api/cron/syncAppMeta] 🔨 Updating app jitsi with categories
@calcom/web:dev: 13:30:47.241 INFO [api/cron/syncAppMeta] 🔨 Updating app around with categories
@calcom/web:dev: 13:30:47.245 INFO [api/cron/syncAppMeta] 🔨 Updating app wipe-my-cal with categories
@calcom/web:dev: 13:30:47.249 INFO [api/cron/syncAppMeta] ✅ App wordpress is up-to-date and correct
@calcom/web:dev: 13:30:47.249 INFO [api/cron/syncAppMeta] 🔨 Updating app huddle01 with categories
@calcom/web:dev: 13:30:47.253 INFO [api/cron/syncAppMeta] 🔨 Updating app riverside with categories
@calcom/web:dev: 13:30:47.257 INFO [api/cron/syncAppMeta] ✅ App ga4 is up-to-date and correct
@calcom/web:dev: 13:30:47.258 INFO [api/cron/syncAppMeta] 🔨 Updating app typeform with categories
@calcom/web:dev: 13:30:47.262 INFO [api/cron/syncAppMeta] 🔨 Updating app webex with categories
@calcom/web:dev: 13:30:47.266 INFO [api/cron/syncAppMeta] 🔨 Updating app ping with categories
@calcom/web:dev: 13:30:47.270 INFO [api/cron/syncAppMeta] ✅ App pipedream is up-to-date and correct
@calcom/web:dev: 13:30:47.270 INFO [api/cron/syncAppMeta] 🔨 Updating app campfire with categories
@calcom/web:dev: 13:30:47.274 INFO [api/cron/syncAppMeta] 🔨 Updating app raycast with categories
@calcom/web:dev: 13:30:47.278 INFO [api/cron/syncAppMeta] 🔨 Updating app sirius_video with categories
@calcom/web:dev: 13:30:47.281 INFO [api/cron/syncAppMeta] ✅ App n8n is up-to-date and correct
@calcom/web:dev: 13:30:47.281 INFO [api/cron/syncAppMeta] ✅ App cron is up-to-date and correct
@calcom/web:dev: 13:30:47.281 INFO [api/cron/syncAppMeta] ✅ App exchange is up-to-date and correct
@calcom/web:dev: 13:30:47.281 INFO [api/cron/syncAppMeta] 🔨 Updating app sendgrid with categories
@calcom/web:dev: 13:30:47.285 INFO [api/cron/syncAppMeta] ✅ App qr_code is up-to-date and correct
@calcom/web:dev: 13:30:47.285 INFO [api/cron/syncAppMeta] ✅ App weather_in_your_calendar is up-to-date and correct
@calcom/web:dev: 13:30:47.285 INFO [api/cron/syncAppMeta] ✅ App gtm is up-to-date and correct
@calcom/web:dev: 13:30:47.285 INFO [api/cron/syncAppMeta] ✅ App fathom is up-to-date and correct
@calcom/web:dev: 13:30:47.285 INFO [api/cron/syncAppMeta] 🔨 Updating app closecom with categories
@calcom/web:dev: 13:30:47.289 INFO [api/cron/syncAppMeta] 🔨 Updating app whatsapp with categories
@calcom/web:dev: 13:30:47.292 INFO [api/cron/syncAppMeta] 🔨 Updating app discord with categories
@calcom/web:dev: 13:30:47.296 INFO [api/cron/syncAppMeta] 🔨 Updating app telegram with categories
@calcom/web:dev: 13:30:47.299 INFO [api/cron/syncAppMeta] 🔨 Updating app signal with categories
@calcom/web:dev: 13:30:47.302 INFO [api/cron/syncAppMeta] 🔨 Updating app sylapsvideo with categories
@calcom/web:dev: 13:30:47.306 INFO [api/cron/syncAppMeta] ✅ App vimcal is up-to-date and correct
@calcom/web:dev: 13:30:47.306 INFO [api/cron/syncAppMeta] ✅ App amie is up-to-date and correct
@calcom/web:dev: 13:30:47.307 INFO [api/cron/syncAppMeta] 🔨 Updating app facetime with categories |
hariombalhara
left a comment
There was a problem hiding this comment.
LGTM now !! Thank you so much @rjackson. I learned a few things from you while collaborating with you on this 🙏
|
@keithwillcode This should be tested on staging. Maybe @ericrommel can do this? |
Yeah no worries! It's been a really fun & interesting task 😆 Sounded like a small job in the first place, but exposed a few Thanks for your feedback and support throughout 🎉 |
|
/tip 50 |
|
tippin a fiffy @rjackson |
|
🎉🎈 @rjackson has been awarded $50! 🎈🎊 |
* Remove unused code in InstalledAppsLayout * Add new app categories "crm", "conferencing" and "messaging" * Sort getAppCategories entries alphabetically * Fix 404s on new category pages (and remove hardcoded category lists) * Fix admin apps list not showing "no available apps" for new categories * Recategorise apps * Sync seed-app-store categories with config files * Replace unnecessary seed-app-store.config.json with appStoreMetadata * Copy video.svg to conferencing.svg * Add messaging.svg * Remove web3 from getAppCategories (used by installed apps, admin apps) * Fix app-store-cli categories - Add conferencing - Add CRM - Remove video - Remove web3 * Remove outdated web3 comment in seed-app-store * Update apps/web/public/static/locales/en/common.json * Add cron script to keep db apps in sync with app metadata * Add redirect for app category "video" to "conferencing" * Fix up "video" category overrides to apply to conferencing * Fix conferencing apps not showing as a location for non-team users * Restore "installed_app" string for conferencing apps * Make linter happier * Remove my "installed_app_conferencing_description" as this was fixed upstream * Quick tidy up * Add dry-run to syncAppMeta via CRON_ENABLE_APP_SYNC env * Replace console.log with logger in syncAppMeta --------- Co-authored-by: Peer Richelsen <peeroke@gmail.com> Co-authored-by: alannnc <alannnc@gmail.com> Co-authored-by: Hariom Balhara <hariombalhara@gmail.com> Co-authored-by: Omar López <zomars@me.com>
What does this PR do?
This PR updates the
config.jsonor_metadata.tsfiles of our app store apps, to recategorise them as per the Google Sheet link provided by in #9304.There are also a number of supporting changes to ensure the app store & categories continue to work correctly after the recategorisation:
seed-app-store.tsfile, so the set of apps that come alongside a dev environment are in-sync with the app metadataseed-app-store.config.jsonwith loading each app's config / metadata entries directly, so its guaranteed to be in-sync with the apps we have in the codebasesyncAppMetacron to keep the App metadata stored in the database (categories, dirName) in sync with the raw config values.Differences from the spreadsheet
There were a few apps in the spreadsheet that were missing from the codebase, which I've therefore not recategorised (because there's nothing to recategorise):
And a few apps that were missing from the spreadsheet, which I've recategorised as:
**Potentially outstanding**This PR does not include any data migrations to correct apps that are already in the database's
Apptable.This does mean that the old categories might still be visible and accessible from the "Apps Store" pages, as those pages infer the supported categories from the apps in the database, rather than using the static list from
getAppsCategoriesor theAppCategoriesenum.Some options we could explore here:
getAppCategoriesand load via app-store metadata, rather than what's in the databaseI don't think this is an immediate issue, because inferring the valid categories from database entries ensures forward/backward compatibility as we recategorise apps.
Fixes #9304
Environment: Staging(main branch) / Production
Type of change
How should this be tested?
Checklist
apps/web/public/app-categorise/messaging.svg(Job for you @PeerRich?)/apps/categories/videoto/apps/categories/conferencing?Solves #9304
/claim #9304