Skip to content

Conversation

@tanuprasad530
Copy link
Contributor

@tanuprasad530 tanuprasad530 commented Dec 11, 2025

  • Added additional use cases of this feature such as leaderboards, showing progression elements (progress bar+rings)
  • Deleted the line that calls out that this includes a feature flag, as it has been enabled for all orgs now.

Closes #523

Summary by CodeRabbit

  • Documentation
    • Updated custom certificates documentation to highlight gamified visual elements (leaderboards, progress rings/bars, milestone progression) that can be integrated into flows.
    • Simplified feature access requirements.

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

- Added additional use cases of this feature such as leaderboards, showing progression elements (progress bar+rings)
- Deleted the line that calls out that this includes a feature flag, as it has been enabled for all orgs now.
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 11, 2025

Walkthrough

This pull request updates the Custom Certificates documentation by adding information about gamified visual elements (leaderboards, progress rings/bars, milestone progression) that can be sent via flows, and removes the prior statement designating the feature as trial-only with beta enrollment requirements.

Changes

Cohort / File(s) Change Summary
Custom Certificates Documentation
docs/4. Product Features/Custom Certificates.md
Added descriptive content about gamified visual elements (leaderboards, progress rings/bars, milestone progression) in the introductory section; removed trial feature designation and beta enrollment requirement language.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

  • No functional code changes; purely documentation prose updates
  • Single file affected with straightforward edits
  • No logic or control-flow implications requiring validation

Possibly related PRs

Suggested reviewers

  • SangeetaMishr
  • mahajantejas

Poem

🐰 A hop and a bound through docs so fine,
Leaderboards sparkle, progress rings align!
No trial chains, just freedom's call,
Gamified glory for one and all!

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Update Custom Certificates.md' is vague and generic, using non-descriptive language that doesn't convey the specific content changes made to the documentation. Consider a more specific title such as 'Add leaderboard and progression examples to Custom Certificates documentation' to better reflect the actual changes.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check ✅ Passed The pull request successfully addresses all coding requirements from issue #523: it adds leaderboard terminology and progression examples to the Custom Certificates documentation as requested.
Out of Scope Changes check ✅ Passed The pull request also removes the feature flag disclaimer, which is mentioned in the PR objectives as a necessary update since the feature is now enabled for all organizations.
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 update-custom_certificates.md

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.

@github-actions
Copy link

@github-actions github-actions bot temporarily deployed to pull request December 11, 2025 04:33 Inactive
Copy link
Contributor

@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: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d1226e1 and 2aba835.

📒 Files selected for processing (1)
  • docs/4. Product Features/Custom Certificates.md (1 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/4. Product Features/Custom Certificates.md

[style] ~21-~21: In American English, abbreviations like “etc.” require a period.
Context: ...ess ring/bar, progression to milestones etc, that can be sent out through flows. ...

(ETC_PERIOD)

🔇 Additional comments (1)
docs/4. Product Features/Custom Certificates.md (1)

15-23: Overview & Pre-requisites integration looks good.

The new sentence at line 21 effectively addresses the PR objectives by introducing leaderboards and progression elements as concrete use cases. The content flows naturally into the Pre-requisites section. The removal of the trial/beta enrollment language simplifies the entry point for users.

Using this feature organizations can upload multiple certificate or badge templates, and share these for the contacts having their personalized information such as "name", or "course participation", "course completion" details or any other such contact specific details be printed on the certificates and have the certificate shared with the contacts through the flows.

## Pre-requisites
In effect, this feature also enables users to create visual gamified elements such as leaderboards, progress ring/bar, progression to milestones etc, that can be sent out through flows.
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Fix punctuation: "etc" should be "etc."

In American English, abbreviations like "etc." require a period. Additionally, consider improving parallel structure for "progress ring/bar" by writing it as "progress bars/rings" or "progress rings/bars" for consistency.

-In effect, this feature also enables users to create visual gamified elements such as leaderboards, progress ring/bar, progression to milestones etc, that can be sent out through flows.
+In effect, this feature also enables users to create visual gamified elements such as leaderboards, progress bars/rings, and progression to milestones, etc., that can be sent out through flows.
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
In effect, this feature also enables users to create visual gamified elements such as leaderboards, progress ring/bar, progression to milestones etc, that can be sent out through flows.
In effect, this feature also enables users to create visual gamified elements such as leaderboards, progress bars/rings, and progression to milestones, etc., that can be sent out through flows.
🧰 Tools
🪛 LanguageTool

[style] ~21-~21: In American English, abbreviations like “etc.” require a period.
Context: ...ess ring/bar, progression to milestones etc, that can be sent out through flows. ...

(ETC_PERIOD)

🤖 Prompt for AI Agents
In docs/4. Product Features/Custom Certificates.md around line 21, fix
punctuation and parallel structure: add a period after "etc" to use "etc." and
normalize the "progress ring/bar" phrase to a consistent parallel form such as
"progress rings/bars" (or "progress bars/rings") so the list reads smoothly and
follows American English conventions.

@Fawas003 Fawas003 merged commit 93de918 into main Dec 16, 2025
7 checks passed
@Fawas003 Fawas003 deleted the update-custom_certificates.md branch December 16, 2025 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update Documentation | Custom Certificates

3 participants