Skip to content

Merge pull request #1 from web/master #12

Open
wants to merge 180 commits into
base: master
Choose a base branch
from
Open

Conversation

ngussed
Copy link

@ngussed ngussed commented Nov 12, 2019

.

aa7401 and others added 25 commits Oct 26, 2019
Merge pull request #1 from web/master
I couldn't make mock-fs work properly, so the unit tests using it don't run. The web server also fails to receive the POST request with the image upload.
One of the modules retained a merge conflict.

This has now been removed.
Async functions can't use the standard readline module.

This has been replaced with the sync version.
@ngussed
Copy link
Author

ngussed commented Nov 12, 2019

s

@aa7401
Copy link
Contributor

aa7401 commented Nov 13, 2019

ease can you explain the changes you want to make and the rationale behind these.

@aa7401 aa7401 added this to Needs review in Kanban Nov 13, 2019
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
Kanban
  
Needs review
Development

Successfully merging this pull request may close these issues.

None yet

9 participants