Skip to content

chore: updated calender links to redirect to same page #2704

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.

Sign up for GitHub

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

Jump to bottom
Merged
ucatbas merged 2 commits into from
Dec 20, 2025
Merged

chore: updated calender links to redirect to same page #2704

ucatbas merged 2 commits into from
Dec 20, 2025
+27 −27

Conversation

@ucatbas
Copy link
Contributor

@ucatbas ucatbas commented Dec 19, 2025 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • Documentation
    • Replaced external scheduling links with Permify's book-demo URL across multiple documentation pages.
    • Corrected misspellings/grammar in help-callout text (e.g., "account executivess" → "account executives").
    • No other content, behavior, or public API changes.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Dec 19, 2025 •
edited
Loading

Walkthrough

This pull request updates help and consultation links across documentation, replacing Calendly scheduling URLs with Permify's booking page at https://www.permify.co/book-demo and fixes several minor typos; no code logic or functionality is changed.

Changes

Cohort / File(s) Summary
Getting Started Examples
docs/getting-started/examples/facebook-groups.mdx, docs/getting-started/examples/google-docs.mdx, docs/getting-started/examples/notion.mdx		 Replaced Calendly links with https://www.permify.co/book-demo in "Need any help?" section
Getting Started Core
docs/getting-started/enforcement.mdx, docs/getting-started/quickstart.mdx, docs/getting-started/testing.mdx		 Replaced Calendly links with https://www.permify.co/book-demo; quickstart updated two links and fixed a typo ("executivess" → "executives")
Installation Guides
docs/setting-up/installation/aws.mdx, docs/setting-up/installation/brew.mdx, docs/setting-up/installation/container.mdx, docs/setting-up/installation/google.mdx		 Replaced Calendly links with https://www.permify.co/book-demo; google.mdx adjusted CTA text/grammar and updated URL
Use Cases
docs/use-cases/abac.mdx, docs/use-cases/custom-roles.mdx, docs/use-cases/multi-tenancy.mdx, docs/use-cases/rbac.mdx, docs/use-cases/rebac.mdx		 Replaced Calendly links with https://www.permify.co/book-demo and corrected typos to "account executives" where present
Overview & Cloud
docs/api-reference/introduction.mdx, docs/cloud/intro.mdx, docs/docs/permify-overview/authorization-service.md, docs/docs/permify-overview/intro.md, docs/docs/playground.md, docs/permify-overview/authorization-service.mdx, docs/permify-overview/intro.mdx		 Replaced Calendly links with https://www.permify.co/book-demo and fixed minor text typos in affected files
Operations & Integrations
docs/operations/cache.mdx, docs/operations/contextual-tuples.mdx, docs/integrations/data-sync.mdx, docs/modeling-guides/rbac/custom-roles.mdx		 Replaced Calendly links with https://www.permify.co/book-demo in help sections

Estimated Code Review Effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Potential focus areas:

  • docs/setting-up/installation/google.mdx — CTA grammar/text change plus URL.
  • Files where both link and typo were fixed (verify only intended text edits).
  • Ensure no accidental whitespace/formatting changes in MDX front-matter.

Possibly related PRs

Poem

🐰 Hopped through docs with nimble feet,
Swapped Calendly links for a Permify meet.
Typos tidied, CTAs now in tune,
Booking page gleams beneath the moon. ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive Title mentions 'calender links' but actual changes involve updating Calendly scheduling URLs to Permify's book-demo page across documentation. The title is vague and contains a typo ('calender' should be 'calendar'), failing to clearly communicate the specific nature of the changes. Revise title to be more specific and accurate, such as 'chore: replace Calendly links with Permify book-demo URLs' and fix the spelling of 'calendar'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/update-calendar-links

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ucatbas ucatbas requested review from EgeAytin and mooreds December 19, 2025 19:48
coderabbitai[bot]
coderabbitai bot reviewed Dec 19, 2025
View reviewed changes
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 10

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3cf7f06 and 977cffb.

📒 Files selected for processing (26)
  • docs/api-reference/introduction.mdx (1 hunks)
  • docs/cloud/intro.mdx (1 hunks)
  • docs/docs/permify-overview/authorization-service.md (1 hunks)
  • docs/docs/permify-overview/intro.md (1 hunks)
  • docs/docs/playground.md (1 hunks)
  • docs/getting-started/enforcement.mdx (1 hunks)
  • docs/getting-started/examples/facebook-groups.mdx (1 hunks)
  • docs/getting-started/examples/google-docs.mdx (1 hunks)
  • docs/getting-started/examples/notion.mdx (1 hunks)
  • docs/getting-started/quickstart.mdx (2 hunks)
  • docs/getting-started/testing.mdx (1 hunks)
  • docs/integrations/data-sync.mdx (1 hunks)
  • docs/modeling-guides/rbac/custom-roles.mdx (1 hunks)
  • docs/operations/cache.mdx (1 hunks)
  • docs/operations/contextual-tuples.mdx (1 hunks)
  • docs/permify-overview/authorization-service.mdx (1 hunks)
  • docs/permify-overview/intro.mdx (1 hunks)
  • docs/setting-up/installation/aws.mdx (1 hunks)
  • docs/setting-up/installation/brew.mdx (1 hunks)
  • docs/setting-up/installation/container.mdx (1 hunks)
  • docs/setting-up/installation/google.mdx (1 hunks)
  • docs/use-cases/abac.mdx (1 hunks)
  • docs/use-cases/custom-roles.mdx (1 hunks)
  • docs/use-cases/multi-tenancy.mdx (1 hunks)
  • docs/use-cases/rbac.mdx (1 hunks)
  • docs/use-cases/rebac.mdx (1 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/operations/cache.mdx

[style] ~98-~98: Consider using a less common alternative to make your writing sound more unique and professional.
Context: ...rchitecture for your permission system. Feel free to [schedule a consultation call with one ...

(FEEL_FREE_TO_STYLE_ME)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
  • GitHub Check: Scan Container for Vulnerabilities
  • GitHub Check: Test with Coverage
  • GitHub Check: Analyze (go)
  • GitHub Check: Test with Coverage
🔇 Additional comments (17)
docs/operations/cache.mdx (1)

98-98: LGTM! Link update is correct.

The consultation link has been successfully updated to Permify's book-demo page.

docs/operations/contextual-tuples.mdx (1)

235-235: LGTM! Link update is correct.

The consultation link has been successfully updated to Permify's book-demo page.

docs/permify-overview/intro.mdx (1)

101-101: LGTM! Link update is correct.

The consultation link has been successfully updated to Permify's book-demo page.

docs/docs/playground.md (1)

160-160: LGTM! Link update is correct.

The consultation link has been successfully updated to Permify's book-demo page.

docs/setting-up/installation/brew.mdx (1)

62-62: LGTM! Link update is correct.

The consultation link has been successfully updated to Permify's book-demo page.

docs/getting-started/examples/notion.mdx (1)

543-543: LGTM!

Link correctly updated to Permify's book-demo page.

docs/setting-up/installation/aws.mdx (1)

185-185: LGTM!

Link correctly updated to Permify's book-demo page.

docs/getting-started/enforcement.mdx (1)

92-92: LGTM!

Link correctly updated to Permify's book-demo page.

docs/getting-started/testing.mdx (1)

304-304: LGTM!

Link correctly updated to Permify's book-demo page.

docs/cloud/intro.mdx (1)

6-6: LGTM!

Link updated correctly to Permify's booking page.

docs/integrations/data-sync.mdx (1)

6-6: LGTM!

Link updated correctly to Permify's booking page.

docs/getting-started/examples/facebook-groups.mdx (1)

540-540: LGTM!

Link updated correctly and text is properly formatted.

docs/setting-up/installation/container.mdx (1)

51-51: LGTM!

Link updated correctly to Permify's booking page.

docs/setting-up/installation/google.mdx (1)

271-271: LGTM!

Link updated correctly, and the grammar fix ("a Permify" instead of "an Permify") is appropriate.

docs/api-reference/introduction.mdx (1)

94-94: LGTM!

The link update from Calendly to the internal booking page is correctly implemented with appropriate link text.

docs/getting-started/quickstart.mdx (1)

19-19: LGTM!

The link update is correctly implemented in the Note section.

docs/getting-started/examples/google-docs.mdx (1)

334-334: LGTM!

The link update is correctly implemented with appropriate context.

@codecov
Copy link

codecov bot commented Dec 19, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.56%. Comparing base (3cf7f06) to head (9c23223).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files 
@@            Coverage Diff             @@
##           master    #2704      +/-   ##
==========================================
+ Coverage   82.55%   82.56%   +0.02%     
==========================================
  Files          74       74              
  Lines        8125     8125              
==========================================
+ Hits         6707     6708       +1     
+ Misses        903      902       -1     
  Partials      515      515

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

coderabbitai[bot]
coderabbitai bot reviewed Dec 19, 2025
View reviewed changes
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Nitpick comments (1)
docs/getting-started/quickstart.mdx (1)

19-19: Link update looks good; consider improving the phrasing.

The URL update to Permify's booking page is consistent with the PR objectives. The phrasing "1x1" is understandable but could be more professional.

💡 Optional: Improve readability

Consider changing "1x1" to "one-on-one" for a more professional tone:

-Want to walk through this guide 1x1 rather than docs ? [schedule a call with a Permify engineer](https://www.permify.co/book-demo).
+Want to walk through this guide one-on-one rather than docs? [schedule a call with a Permify engineer](https://www.permify.co/book-demo).

(Also removed the extra space before the question mark for consistency.)

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 977cffb and 9c23223.

📒 Files selected for processing (10)
  • docs/docs/permify-overview/authorization-service.md (1 hunks)
  • docs/docs/permify-overview/intro.md (1 hunks)
  • docs/getting-started/quickstart.mdx (2 hunks)
  • docs/modeling-guides/rbac/custom-roles.mdx (1 hunks)
  • docs/permify-overview/authorization-service.mdx (1 hunks)
  • docs/use-cases/abac.mdx (1 hunks)
  • docs/use-cases/custom-roles.mdx (1 hunks)
  • docs/use-cases/multi-tenancy.mdx (1 hunks)
  • docs/use-cases/rbac.mdx (1 hunks)
  • docs/use-cases/rebac.mdx (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (8)
  • docs/docs/permify-overview/authorization-service.md
  • docs/use-cases/custom-roles.mdx
  • docs/use-cases/abac.mdx
  • docs/use-cases/multi-tenancy.mdx
  • docs/use-cases/rbac.mdx
  • docs/docs/permify-overview/intro.md
  • docs/permify-overview/authorization-service.mdx
  • docs/use-cases/rebac.mdx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
  • GitHub Check: Test with Coverage
  • GitHub Check: Analyze (go)
  • GitHub Check: Scan Container for Vulnerabilities
  • GitHub Check: Test with Coverage
🔇 Additional comments (2)
docs/modeling-guides/rbac/custom-roles.mdx (1)

126-126: LGTM! Link updated and typo fixed.

Both the calendar link has been successfully updated to Permify's book-demo page and the typo "executivess" → "executives" has been corrected.

docs/getting-started/quickstart.mdx (1)

250-250: Excellent! Both the link update and typo fix have been applied.

The URL has been correctly updated to Permify's booking page, and the typo "executivess" has been fixed to "executives" as requested in the previous review. These changes improve both functionality and professionalism of the documentation.

mooreds
mooreds approved these changes Dec 19, 2025
View reviewed changes
Copy link
Contributor

@mooreds mooreds 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 to me.

@ucatbas ucatbas merged commit 662d932 into master Dec 20, 2025
15 checks passed
@ucatbas ucatbas deleted the chore/update-calendar-links branch December 20, 2025 03:53
@coderabbitai coderabbitai bot mentioned this pull request Jan 6, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

@mooreds mooreds mooreds approved these changes

@EgeAytin EgeAytin Awaiting requested review from EgeAytin

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ucatbas @mooreds