Skip to content

Conversation

@rhen92
Copy link
Collaborator

@rhen92 rhen92 commented Jan 6, 2026

Overview

🎟 Relevant Jira Issues

📚 What is the context and goal of this PR?

🥴 TL; RL:

💡 Feature Breakdown (screenshots & videos encouraged!)

🛠 Important tradeoffs made:

🔍 Types of Changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Chore (refactor, documentation update, etc)

💳 Does This Create Any New Technical Debt? ( If yes, please describe and add JIRA TODOs )

  • No
  • Yes

Testing

🔬 How Can Someone QA This?

📱 🖥 Which devices would you like help testing on?

🧪 Code Coverage

Documentation

📝 Documentation Checklist

User-Facing Docs (docs/docs.learncard.com)

  • Tutorial — New capability that users need to learn (docs/tutorials/)
  • How-To Guide — New workflow or integration (docs/how-to-guides/)
  • Reference — New/changed API, config, or SDK method (docs/sdks/)
  • Concept — New mental model or architecture explanation (docs/core-concepts/)
  • App Flows — Changes to LearnCard App or ScoutPass user flows (docs/apps/)

Internal/AI Docs

  • CLAUDE.md — New pattern, flow, or context that AI assistants need
  • Code comments/JSDoc — Complex logic that needs inline explanation

Visual Documentation

  • Mermaid diagram — Complex flow, state machine, or architecture

💭 Documentation Notes

✅ PR Checklist

  • Related to a Jira issue (create one if not)
  • My code follows style guidelines (eslint / prettier)
  • I have manually tested common end-2-end cases
  • I have reviewed my code
  • I have commented my code, particularly where ambiguous
  • New and existing unit tests pass locally with my changes
  • I have completed the Documentation Checklist above (or explained why N/A)

🚀 Ready to squash-and-merge?:

  • Code is backwards compatible
  • There is not a "Do Not Merge" label on this PR
  • I have thoughtfully considered the security implications of this change.
  • This change does not expose new public facing endpoints that do not have authentication

@changeset-bot
Copy link

changeset-bot bot commented Jan 6, 2026

⚠️ No Changeset found

Latest commit: 4792c87

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@netlify
Copy link

netlify bot commented Jan 6, 2026

Deploy Preview for staging-learncardapp canceled.

Name Link
🔨 Latest commit 4792c87
🔍 Latest deploy log https://app.netlify.com/projects/staging-learncardapp/deploys/69605d6a511ffe000816ec42

@netlify
Copy link

netlify bot commented Jan 6, 2026

Deploy Preview for learncarddocs canceled.

Name Link
🔨 Latest commit 4792c87
🔍 Latest deploy log https://app.netlify.com/projects/learncarddocs/deploys/69605d6aa7a2090008c64aca

@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

👋 Hey there! It looks like you modified code, but didn't update the documentation in /docs.

If this PR introduces new features, changes APIs, or modifies behavior that users or developers need to know about, please consider updating the docs.


🏄 Windsurf Tip

You can ask Windsurf to help:

"Analyze the changes in this PR and update the gitbook docs in /docs accordingly."

Windsurf will review your changes and suggest appropriate documentation updates based on what was modified.


📚 Documentation Guide
Change Type Doc Location
New feature/API docs/tutorials/ or docs/how-to-guides/
SDK/API changes docs/sdks/
New concepts docs/core-concepts/
App UI/UX flows docs/apps/ (LearnCard App, ScoutPass)
Internal patterns CLAUDE.md

This is an automated reminder. If no docs are needed, feel free to ignore this message.

@TaylorBeeston TaylorBeeston temporarily deployed to learn-card-app-staging January 6, 2026 21:37 — with GitHub Actions Inactive
@rhen92 rhen92 temporarily deployed to learn-cloud-network-api-staging January 7, 2026 21:41 — with GitHub Actions Inactive
@rhen92 rhen92 temporarily deployed to learn-card-app-staging January 7, 2026 21:45 — with GitHub Actions Inactive
@rhen92 rhen92 temporarily deployed to learn-cloud-network-api-staging January 9, 2026 00:47 — with GitHub Actions Inactive
@rhen92 rhen92 temporarily deployed to learn-card-app-staging January 9, 2026 00:50 — with GitHub Actions Inactive
@rhen92 rhen92 temporarily deployed to learn-cloud-network-api-staging January 9, 2026 01:24 — with GitHub Actions Inactive
@rhen92 rhen92 temporarily deployed to learn-card-app-staging January 9, 2026 01:27 — with GitHub Actions Inactive
@rhen92 rhen92 temporarily deployed to learn-cloud-network-api-staging January 9, 2026 01:44 — with GitHub Actions Inactive
@rhen92 rhen92 deployed to learn-card-app-staging January 9, 2026 01:48 — with GitHub Actions Active
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.

3 participants