Skip to content

Removed the Contact link from header and footer#214

Open
aalfee wants to merge 1 commit into
mainfrom
remove/contact
Open

Removed the Contact link from header and footer#214
aalfee wants to merge 1 commit into
mainfrom
remove/contact

Conversation

@aalfee

@aalfee aalfee commented Sep 3, 2025

Copy link
Copy Markdown
Contributor

In directory /app/components/header/:

Removed the link to Contact page in navitems.ts.

In directory /components/:

Removed the link to Contact page in components-footer.tsx.

@aalfee aalfee requested a review from a team as a code owner September 3, 2025 19:34
@vercel

vercel Bot commented Sep 3, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
flushingtech-org Ready Ready Preview Comment Sep 3, 2025 7:34pm

@benhuangbmj benhuangbmj left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please resolve the code conflicts.

@wilrnh wilrnh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is there any particular reason the nextjs version was bumped? If not, could we revert the package.json changes?

@aalfee

aalfee commented Sep 17, 2025

Copy link
Copy Markdown
Contributor Author

Updates for package.json and package-lock.json files are automatic version updates pushed with the changes. Let me know if you want me to reverse only the next version update? Or all the automatic version updates?

Comment thread package-lock.json

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the clarification. Yea I'd say reverse all of them, unless you think any are needed for this PR.

I'm also curious when the updates are done - is it when npm run build or npm run lint is done?

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.

4 participants