search

gregrickaby / nextjs-wordpress

💀 It's headless WordPress!
https://nextjswp.com
MIT License
246 stars 50 forks source link

Auto install setup - wrong url link and port 80 in use #74

Closed Frosty21 closed 1 year ago

Frosty21 commented 1 year ago

Description

This changes the port from 80 to 81 in container "nextjs-wordpress-traefik" as I find with some devs it will have port 80 already in use (open to change it to another port if needed) reference traefik needs port 80. Also the url link in install.sh and setup.sh needed a '/' so i've added that into https://nextjswp.test/wp-admin/

Screenshot

https://user-images.githubusercontent.com/5212703/222613018-3255894f-2b35-4c79-8c92-f26527a4265b.mov

Screen Shot 2023-03-02 at 7 32 56 PM

Verification

How will a code reviewer test this?

  1. Run through the auto install process npm run install
  2. Follow the prompts
  3. Go to https://nextjswp.test/wp-admin/ that it works
vercel[bot] commented 1 year ago

Someone is attempting to deploy a commit to a Personal Account owned by @gregrickaby on Vercel.

@gregrickaby first needs to authorize it.

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
nextjs-wordpress ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 3, 2023 at 1:10PM (UTC)
gregrickaby commented 1 year ago

@Frosty21 Thank you so much and Go Pack Go! 🏈

Frosty21 commented 1 year ago

@gregrickaby No worries! Hopefully the Packs make it to the playoffs this season 🙏