Remove Redundant Directories - Issue#15

dgghun / LawyerUp

Video chat application

MIT License

1 stars 0 forks source link

Remove Redundant Directories - Issue#15 #17

Closed VonHohenheim closed 5 years ago

VonHohenheim commented 5 years ago

Deleted the following files: ./public/ ./video_chat.html ./js/app.js ./now.sh

VonHohenheim commented 5 years ago

If there's no issues with the app running on the server then you should do a Rebase and merge instead of a Merge pull request. Reason we used the merge pull request last time was because of the commit issues. Using the rebase option this time won't add a "merge commit" onto the branch.

dgghun commented 5 years ago

Added the "delete the /public directory" on accident. We need to keep that one to maintain the video chat portion. Also, I still see the ./js/app.js and ./now.sh files in the #15 branch? So the others are okay to remove just not the /public directory. Make sense? Sorry, I am going on negative sleep and brain is fuzzy.

VonHohenheim commented 5 years ago

Closed pull request with merge due to deleted ./public folder. Will resubmit request.