etesync-notes icon indicating copy to clipboard operation
etesync-notes copied to clipboard

following README's dev instructions results fails (at least for me)

Open jzacsh opened this issue 3 years ago • 3 comments

yarn web results in:

yarn run v1.22.11
$ expo start --web
Starting project at /tmp/src/etesync-notes
Unable to find expo in this project - have you run yarn / npm install yet?
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

note i've had yarn run successfully, as well as npm install (trying to follow the above error's expo hint)

$ npm --version && yarn --version && expo-cli --version
7.20.3
1.22.11
4.9.1

possible lead: is it correct that the READMe says to have both yarn and npm installed?

tdl;dr maybe this is a "support" ticket (in which case, feel free to close it - i'm happy to try debugging live in the matrix room instead) but maybe something's off with the readme? wanna double-check before i investigate further

jzacsh avatar Aug 01 '21 01:08 jzacsh

npm install may have littered your env.

Start from a fresh directory and follow the prerequisites - https://github.com/etesync/etesync-notes#prerequisites

I think the issue you are facing (and we should probably update the readme!) is that developing the web requires yarn add expo. We are however not adding this dep permanently because it adds non-free components to the Android and iOS bulids which we don't want.

tasn avatar Aug 01 '21 09:08 tasn

Thanks, that worked and indeed I'm able to see the UI on localhost now.

I've started a pull request with the update to the readme you mention, and threw in a few other TODOs that might be helpful for me to figure out and include before I close that PR.

jzacsh avatar Aug 01 '21 23:08 jzacsh

Great to hear! Cool, let me know once the PR is ready to review.

tasn avatar Aug 02 '21 08:08 tasn