Skip to content

Configure to use dependency cooldowns#342

Open
chiaberry wants to merge 2 commits into21517-app-nextfrom
chia/27715-depend-cooldown
Open

Configure to use dependency cooldowns#342
chiaberry wants to merge 2 commits into21517-app-nextfrom
chia/27715-depend-cooldown

Conversation

@chiaberry
Copy link
Copy Markdown
Member

@chiaberry chiaberry commented Apr 14, 2026

cityofaustin/atd-data-tech#27715

https://deploy-preview-342--nextjs-knack-signs-markings.netlify.app/

This app is in the /signs-work-order-map directory.

  1. cd signs-work-order-map
  2. Delete your node_modules folder.
  3. Run nvm use.
  4. If you did not test VZ PR 2005, Run npm install -g npm@latest
  5. Check your npm version and verify you're on 11.12.1: npm -v
  6. npm install
  7. Run npm run dev

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 14, 2026

Deploy Preview for nextjs-knack-signs-markings ready!

Name Link
🔨 Latest commit 0e1984c
🔍 Latest deploy log https://app.netlify.com/projects/nextjs-knack-signs-markings/deploys/69dea6dee37fea0008f7744f
😎 Deploy Preview https://deploy-preview-342--nextjs-knack-signs-markings.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@chiaberry chiaberry changed the base branch from staging to 21517-app-next April 14, 2026 20:43
@chiaberry chiaberry requested a review from mateoclarke April 14, 2026 20:43
@chiaberry
Copy link
Copy Markdown
Member Author

@mateoclarke I was thinking of bumping the NextJS version as well, what do you think?

Asking for review from you first since you are actively working on this, want to make sure i dont disrupt any of your work in progress

Copy link
Copy Markdown
Contributor

@mateoclarke mateoclarke left a comment

Choose a reason for hiding this comment

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

I would update the testing steps to let folks know specifically which node_modules folder needs to be deleted. I think it should be /signs-work-order-map/node_modules. Also, wouldn't hurt to tell testers to cd signs-work-order-map before running nvm use.

I think its a good idea to upgrade Next, but maybe in a different PR. Thanks for checking to see if this or the upgrade would be disruptive to my work on this but I feel like its fine and better to upgrade sooner than later. No conflicts afaik.

Copy link
Copy Markdown
Collaborator

@mddilley mddilley left a comment

Choose a reason for hiding this comment

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

Followed the test steps and the readme, and I was up and running with the node 24 and npm 11.13.0 🚢 🙏

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