-
Notifications
You must be signed in to change notification settings - Fork 1
PR | Hanko Remix Starter | Updated Starter -> Main #1
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
Did some minor changes
LICENSE
Outdated
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.
The license should not be deleted,
app/routes/dashboard.tsx
Outdated
<HankoProfile /> | ||
</div> | ||
) | ||
const dashboardPage = () => { |
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.
Please use uppercase for the first letter, as the other components.
const dashboardPage = () => { | |
const DashboardPage = () => { |
app/routes/profile.tsx
Outdated
return { hankoUrl: hankoUrl }; | ||
}; | ||
|
||
const profilePage = () => { |
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.
Please use uppercase for the first letter, as the other components.
const profilePage = () => { | |
const ProfilePage = () => { |
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.
Fixed your feedback in the latest commit
Readded the License and changed letters for page components
My pull request for the Hanko Remix starter.
If theres anything that needs to be changed lmk and Ill look into it!