-
Notifications
You must be signed in to change notification settings - Fork 28
Merge button for PR #52
Comments
Hey I would like to take this one, but before that, I have some issues with running your project on iOS simulator, I keep getting errors about wrong import paths, eg. you import something from 'actions', which is one folder up, so if I replace it with '../actions' it works, but I don;t want to do it for every import statement :) so how can I keep it as in your code but without errors? Thanks! |
@brascene did you start packer manually by |
@ovr no, I started it with react-native start |
@brascene please use Thank |
@ovr thank you, I'll check it later today and start with button :) one quicky: what PR screen stands for? |
@brascene PR = Pull Request, you can open any pull request from the Feed (it's main screen after login) |
@ovr thanks, I was thinking maybe it's some shortcut for one of your components or screens. I've did what you said about running packager with npm run start, but now I receive another issue, and that is:
|
You can ignore this error, it's not a problem
Looks like this problem comes in latest commits in master, I am going to fix it |
Hey!
We needed button for merge inside on PR page :)
THank
The text was updated successfully, but these errors were encountered: