Skip to content
This repository was archived by the owner on Mar 17, 2026. It is now read-only.

docs: updates primary button to sign in#89

Merged
jyecusch merged 1 commit into
nitrictech:mainfrom
davemooreuws:add-signin-link
Sep 26, 2025
Merged

docs: updates primary button to sign in#89
jyecusch merged 1 commit into
nitrictech:mainfrom
davemooreuws:add-signin-link

Conversation

@davemooreuws
Copy link
Copy Markdown
Member

No description provided.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Sep 26, 2025

Walkthrough

Updates docs/docs.json to modify the primary navbar call-to-action. The label changes from "Sign up" to "Sign in", and the link updates from https://www.addsuga.com/#early-access to https://app.addsuga.com/login. No exported or public entity declarations are affected.

Possibly related PRs

Suggested reviewers

  • jyecusch
  • tjholm
  • HomelessDinosaur

Pre-merge checks

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning The pull request lacks any description, providing no context or explanation for the change, which fails to relate to the modifications made in the docs.json file. Please add a brief description explaining that the primary navbar CTA was changed from “Sign up” to “Sign in” and the link was updated to the login URL to give reviewers necessary context.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The title clearly and concisely summarizes the main change by indicating that the primary button in the docs has been updated to “Sign in,” matching the actual modifications in the JSON file. It’s short, specific, and directly relates to the core update without unnecessary detail. This meets the criteria for a focused, informative pull request title.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 97a5a74 and 0ddcdf7.

📒 Files selected for processing (1)
  • docs/docs.json (1 hunks)
🔇 Additional comments (1)
docs/docs.json (1)

133-134: CTA update looks good

New label and destination match the sign-in flow goal without impacting other navbar config.


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.

@jyecusch jyecusch merged commit c33bc3f into nitrictech:main Sep 26, 2025
4 checks passed
@nitric-bot
Copy link
Copy Markdown

🎉 This PR is included in version 0.1.8 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants