StockAlertBot icon indicating copy to clipboard operation
StockAlertBot copied to clipboard

Notification Testing & Dockerization

Open ryand11 opened this issue 2 years ago • 1 comments

Describe the bug

  1. In Current build, there is no way to test to see if a webhook or SMS is working outside of adding an already in-stock product
  2. After some gymnastics, I was able to dockerize the application with a tweak to your SERVER_URL Parm in index.html. Can this be moved to an .env variable so that this can be hosted on a server? ( I have attached the dockerfile if you'd like to add to your repo)

Logs and Screenshots No Logs

Platform Information

  • OS: buster aarch64
  • Node version : 13
Screen Shot 2021-08-22 at 11 26 47 AM

ryand11 avatar Aug 22 '21 15:08 ryand11

Hi there!

Just to make sure I'm understanding this correctly, you're not reporting a bug in the program. You're just suggesting that I:

  1. Add a way to test if webhook or SMS are working.
  2. Make SERVER_ENDPOINT be editable in .env.

If so, then I prefer if you post these suggestions in the discussions so I can track them easier. Number 1 is a good idea and I'll add it to the list of things to do. Number 2, I think it makes more sense to add it to config.json.

Prince25 avatar Aug 23 '21 20:08 Prince25