-
Notifications
You must be signed in to change notification settings - Fork 65
fix: links to examples in examples #579
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
base: main
Are you sure you want to change the base?
Conversation
|
|
@emliunix is attempting to deploy a commit to the ory Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates GitHub repository URLs to reflect the branch name change from "master" to "main" and corrects the paths to point to the appropriate example directories. The changes ensure that links to the App Router and Pages Router examples are accurate and use the current default branch name.
Key changes:
- Updated branch name from "master" to "main" in GitHub URLs
- Fixed incorrect paths to ensure each example links to the correct directory
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| examples/nextjs-pages-router/pages/index.tsx | Updated GitHub URLs to use "main" branch and corrected the App Router example path |
| examples/nextjs-app-router/app/page.tsx | Updated GitHub URLs to use "main" branch and corrected the App Router example path |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #579 +/- ##
===========================================
+ Coverage 42.43% 54.47% +12.04%
===========================================
Files 136 159 +23
Lines 2008 3042 +1034
Branches 288 461 +173
===========================================
+ Hits 852 1657 +805
- Misses 1149 1323 +174
- Partials 7 62 +55
🚀 New features to boost your workflow:
|
vinckr
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks! lgtm ![]()
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
The index pages in examples contains outdated self links to examples.
Related Issue or Design Document
n/a
Checklist
If this pull request addresses a security vulnerability,
I confirm that I got approval (please contact [email protected]) from the maintainers to push the changes.