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

chore: move calcom/docs into mono repo #14906

Closed
wants to merge 2 commits into from

Conversation

anikdhabal
Copy link
Contributor

@anikdhabal anikdhabal commented May 6, 2024

What does this PR do?

Fixes #14772

Requirement/Documentation

  • If there is a requirement document, please, share it here.
  • If there is a UI/UX design document, please, share it here.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Chore (refactoring code, technical debt, workflow improvements)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Tests (Unit/Integration/E2E or any other test)
  • This change requires a documentation update

How should this be tested?

  • Are there environment variables that should be set?
  • What are the minimal test data to have?
  • What is expected (happy path) to have (input and output)?
  • Any other important info that could help to test that PR

Mandatory Tasks

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

Checklist

  • I haven't read the contributing guide
  • My code doesn't follow the style guidelines of this project
  • I haven't commented my code, particularly in hard-to-understand areas
  • I haven't checked if my PR needs changes to the documentation
  • I haven't checked if my changes generate no new warnings
  • I haven't added tests that prove my fix is effective or that my feature works
  • I haven't checked if new and existing unit tests pass locally with my changes

Copy link

vercel bot commented May 6, 2024

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

A member of the Team first needs to authorize it.

@graphite-app graphite-app bot added the community Created by Linear-GitHub Sync label May 6, 2024
@graphite-app graphite-app bot requested review from a team May 6, 2024 22:45
Copy link
Contributor

github-actions bot commented May 6, 2024

Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link.

@dosubot dosubot bot added the docs area: docs, documentation, cal.com/docs label May 6, 2024
Copy link

socket-security bot commented May 6, 2024

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/next@11.1.4 environment, filesystem, network, shell, unsafe Transitive: eval +301 188 MB vercel-release-bot
npm/nextra-theme-docs@1.2.6 network Transitive: environment, eval, filesystem, shell, unsafe +500 47.1 MB quietshu
npm/nextra@1.1.0 environment, shell Transitive: eval, filesystem, network +219 6.64 MB quietshu
npm/prettier@2.8.8 environment, filesystem, unsafe 0 11.2 MB prettier-bot
npm/react-dom@17.0.2 environment +1 3.1 MB gaearon

🚮 Removed packages: npm/@calcom/ai@0.0.0-use.local, npm/@calcom/alby@0.0.0-use.local, npm/@calcom/amie@0.0.0-use.local, npm/@calcom/api-proxy@0.0.0-use.local, npm/@calcom/api-v2@0.0.0-use.local, npm/@calcom/api@0.0.0-use.local, npm/@calcom/app-store-cli@0.0.0-use.local, npm/@calcom/app-store@0.0.0-use.local, npm/@calcom/applecalendar@0.0.0-use.local, npm/@calcom/around@0.0.0-use.local, npm/@calcom/atoms@0.0.0-use.local, npm/@calcom/atoms@1.0.40, npm/@calcom/autocheckin@0.0.0-use.local, npm/@calcom/base@0.0.0-use.local, npm/@calcom/basecamp3@0.0.0-use.local, npm/@calcom/cal-ai@0.0.0-use.local, npm/@calcom/caldavcalendar@0.0.0-use.local, npm/@calcom/campfire@0.0.0-use.local, npm/@calcom/clic@0.0.0-use.local, npm/@calcom/closecom@0.0.0-use.local, npm/@calcom/config@0.0.0-use.local, npm/@calcom/console@0.0.0-use.local, npm/@calcom/core@0.0.0-use.local, npm/@calcom/cron@0.0.0-use.local, npm/@calcom/dailyvideo@0.0.0-use.local, npm/@calcom/dayjs@0.0.0-use.local, npm/@calcom/debugging@0.0.0-use.local, npm/@calcom/deel@0.0.0-use.local, npm/@calcom/demodesk@0.0.0-use.local, npm/@calcom/discord@0.0.0-use.local, npm/@calcom/ee@0.0.0-use.local, npm/@calcom/eightxeight@0.0.0-use.local, npm/@calcom/element-call@0.0.0-use.local, npm/@calcom/emails@0.0.0-use.local, npm/@calcom/embed-core@0.0.0-use.local, npm/@calcom/embed-react@0.0.0-use.local, npm/@calcom/embed-snippet@0.0.0-use.local, npm/@calcom/eslint-plugin-eslint@0.0.0-use.local, npm/@calcom/example-app-credential-sync@0.0.0-use.local, npm/@calcom/exchange2013calendar@0.0.0-use.local, npm/@calcom/exchange2016calendar@0.0.0-use.local, npm/@calcom/exchangecalendar@0.0.0-use.local, npm/@calcom/facetime@0.0.0-use.local, npm/@calcom/fathom@0.0.0-use.local, npm/@calcom/feature-auth@0.0.0-use.local, npm/@calcom/features@0.0.0-use.local, npm/@calcom/feishucalendar@0.0.0-use.local, npm/@calcom/ga4@0.0.0-use.local, npm/@calcom/giphy@0.0.0-use.local, npm/@calcom/googlecalendar@0.0.0-use.local, npm/@calcom/googlevideo@0.0.0-use.local, npm/@calcom/gtm@0.0.0-use.local, npm/@calcom/hubspot@0.0.0-use.local, npm/@calcom/huddle01video@0.0.0-use.local, npm/@calcom/ics-feed@0.0.0-use.local, npm/@calcom/intercom@0.0.0-use.local, npm/@calcom/jelly@0.0.0-use.local, npm/@calcom/jitsivideo@0.0.0-use.local, npm/@calcom/kysely@0.0.0-use.local, npm/@calcom/larkcalendar@0.0.0-use.local, npm/@calcom/lib@0.0.0-use.local, npm/@calcom/linear@0.0.0-use.local, npm/@calcom/make@0.0.0-use.local, npm/@calcom/matomo@0.0.0-use.local, npm/@calcom/metapixel@0.0.0-use.local, npm/@calcom/mirotalk@0.0.0-use.local, npm/@calcom/mock-payment-app@0.0.0-use.local, npm/@calcom/n8n@0.0.0-use.local, npm/@calcom/office365calendar@0.0.0-use.local, npm/@calcom/office365video@0.0.0-use.local, npm/@calcom/paypal@0.0.0-use.local, npm/@calcom/ping@0.0.0-use.local, npm/@calcom/pipedream@0.0.0-use.local, npm/@calcom/pipedrive-crm@0.0.0-use.local, npm/@calcom/platform-constants@0.0.0-use.local, npm/@calcom/platform-libraries@0.0.0-use.local, npm/@calcom/platform-types@0.0.0-use.local, npm/@calcom/platform-utils@0.0.0-use.local, npm/@calcom/plausible@0.0.0-use.local, npm/@calcom/prisma@0.0.0-use.local, npm/@calcom/qr_code@0.0.0-use.local, npm/@calcom/raycast@0.0.0-use.local, npm/@calcom/riverside@0.0.0-use.local, npm/@calcom/roam@0.0.0-use.local, npm/@calcom/routing-forms@0.0.0-use.local, npm/@calcom/salesforce@0.0.0-use.local, npm/@calcom/salesroom@0.0.0-use.local, npm/@calcom/sdk@0.0.0-use.local, npm/@calcom/sendgrid@0.0.0-use.local, npm/@calcom/shimmer-video@0.0.0-use.local, npm/@calcom/signal@0.0.0-use.local, npm/@calcom/sirius_video@0.0.0-use.local, npm/@calcom/storybook@0.0.0-use.local, npm/@calcom/stripepayment@0.0.0-use.local, npm/@calcom/swagger@0.0.0-use.local, npm/@calcom/sylapsvideo@0.0.0-use.local, npm/@calcom/tandemvideo@0.0.0-use.local, npm/@calcom/telegram@0.0.0-use.local, npm/@calcom/trpc@0.0.0-use.local, npm/@calcom/tsconfig@0.0.0-use.local, npm/@calcom/typeform@0.0.0-use.local, npm/@calcom/types@0.0.0-use.local, npm/@calcom/ui@0.0.0-use.local, npm/@calcom/vimcal@0.0.0-use.local, npm/@calcom/vital@0.0.0-use.local, npm/@calcom/weather_in_your_calendar@0.0.0-use.local, npm/@calcom/web@0.0.0-use.local, npm/@calcom/webex@0.0.0-use.local, npm/@calcom/website@0.0.0-use.local, npm/@calcom/whatsapp@0.0.0-use.local, npm/@calcom/whereby@0.0.0-use.local, npm/@calcom/wordpress@0.0.0-use.local, npm/@calcom/zapier@0.0.0-use.local, npm/@calcom/zoho-bigin@0.0.0-use.local, npm/@calcom/zohocalendar@0.0.0-use.local, npm/@calcom/zohocrm@0.0.0-use.local, npm/@calcom/zoomvideo@0.0.0-use.local, npm/@types/node@16.9.1, npm/@types/node@20.12.8, npm/@types/react@18.0.26, npm/@upstash/redis@1.30.1, npm/calcom-monorepo@0.0.0-use.local, npm/css-what@6.1.0, npm/micromark-util-html-tag-name@2.0.0, npm/rrule@2.8.1, npm/superjson@1.9.1, npm/wipemycal@0.0.0-use.local

View full report↗︎

Copy link

socket-security bot commented May 6, 2024

🚨 Potential security issues detected. Learn more about Socket for GitHub ↗︎

To accept the risk, merge this PR and you will not be notified again.

Alert Package NoteSource
Install scripts npm/fsevents@1.2.13
  • Install script: install
  • Source: node install.js
Telemetry npm/next@11.1.4
  • Note: Can be disabled by setting the environment variable NEXT_TELEMETRY_DISABLED=1 . See https://nextjs.org/telemetry for more information
Install scripts npm/@ampproject/toolbox-optimizer@2.6.0
  • Install script: postinstall
  • Source: node lib/warmup.js
Telemetry npm/next@9.5.5
  • Note: Can be disabled by setting the environment variable NEXT_TELEMETRY_DISABLED=1 . See https://nextjs.org/telemetry for more information
Protestware/Troll package npm/es5-ext@0.10.64
  • Note: This package prints a protestware console message on install regarding Ukraine for users with Russian language locale
Install scripts npm/es5-ext@0.10.64
  • Install script: postinstall
  • Source: node -e "try{require('./_postinstall')}catch(e){}" || exit 0

View full report↗︎

Next steps

What is an install script?

Install scripts are run when the package is installed. The majority of malware in npm is hidden in install scripts.

Packages should not be running non-essential scripts during install and there are often solutions to problems people solve with install scripts that can be run at publish time instead.

What is telemetry?

This package contains telemetry which tracks how it is used.

Most telemetry comes with settings to disable it. Consider disabling telemetry if you do not want to be tracked.

What is protestware?

This package is a joke, parody, or includes undocumented or hidden behavior unrelated to its primary function.

Consider that consuming this package my come along with functionality unrelated to its primary purpose.

Take a deeper look at the dependency

Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support [AT] socket [DOT] dev.

Remove the package

If you happen to install a dependency that Socket reports as Known Malware you should immediately remove it and select a different dependency. For other alert types, you may may wish to investigate alternative packages or consider if there are other ways to mitigate the specific risk posed by the dependency.

Mark a package as acceptable risk

To ignore an alert, reply with a comment starting with @SocketSecurity ignore followed by a space separated list of ecosystem/package-name@version specifiers. e.g. @SocketSecurity ignore npm/foo@1.0.0 or ignore all packages with @SocketSecurity ignore-all

  • @SocketSecurity ignore npm/fsevents@1.2.13
  • @SocketSecurity ignore npm/next@11.1.4
  • @SocketSecurity ignore npm/@ampproject/toolbox-optimizer@2.6.0
  • @SocketSecurity ignore npm/next@9.5.5
  • @SocketSecurity ignore npm/es5-ext@0.10.64

Copy link

graphite-app bot commented May 6, 2024

Graphite Automations

"Add community label" took an action on this PR • (05/06/24)

1 label was added to this PR based on Keith Williams's automation.

"Add foundation team as reviewer" took an action on this PR • (05/06/24)

1 reviewer was added to this PR based on Keith Williams's automation.

"Add consumer team as reviewer" took an action on this PR • (05/06/24)

1 reviewer was added to this PR based on Keith Williams's automation.

@github-actions github-actions bot added foundation Medium priority Created by Linear-GitHub Sync labels May 6, 2024
Copy link
Contributor

github-actions bot commented May 6, 2024

📦 Next.js Bundle Analysis for @calcom/web

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

Seven Pages Changed Size

The following pages changed size from the code in this PR compared to its base branch:

Page Size (compressed) First Load % of Budget (350 KB)
/event-types 556.73 KB 785.3 KB 224.37% (🟡 +0.18%)
/getting-started/[[...step]] 405.19 KB 633.76 KB 181.08% (🟡 +0.18%)
/settings/my-account/profile 406.02 KB 634.6 KB 181.31% (🟡 +0.19%)
/settings/organizations/profile 398.79 KB 627.37 KB 179.25% (🟡 +0.18%)
/settings/organizations/teams/other/[id]/profile 469.2 KB 697.77 KB 199.36% (🟡 +0.19%)
/settings/teams/[id]/profile 470.03 KB 698.6 KB 199.60% (🟡 +0.18%)
/workflows/[workflow] 410.63 KB 639.21 KB 182.63% (🟡 +0.18%)
Details

Only 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 next/link is used, subsequent page loads would only need to download that page's bundle (the number in the "Size" column), since the global bundle has already been downloaded.

Any third party scripts you have added directly to your app using the <script> tag are not accounted for in this analysis

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.

@keithwillcode keithwillcode added this to the v4.1 milestone May 8, 2024
@dosubot dosubot bot modified the milestones: v4.1, v4.2 May 9, 2024
@keithwillcode
Copy link
Contributor

Thanks for this @anikdhabal but unfortunately we weren't able to get it merged quickly enough and now there have been structural changes to docs. We'll need to redo this PR. We can take it internally to make sure it's done in 1 motion and nothing is lost in the move.

@dosubot dosubot bot modified the milestone: v4.2 May 23, 2024
@anikdhabal
Copy link
Contributor Author

Thanks for this @anikdhabal but unfortunately we weren't able to get it merged quickly enough and now there have been structural changes to docs. We'll need to redo this PR. We can take it internally to make sure it's done in 1 motion and nothing is lost in the move.

Sure Keith, understandable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Created by Linear-GitHub Sync docs area: docs, documentation, cal.com/docs foundation Medium priority Created by Linear-GitHub Sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CAL-3563] move calcom/docs into mono repo
2 participants