Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Updating #119

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

Updating #119

wants to merge 64 commits into from

Conversation

Mortuie
Copy link
Contributor

@Mortuie Mortuie commented Oct 6, 2018

No description provided.

nikrb and others added 30 commits February 10, 2018 23:37
@nikrb
add backend user model unit test
b1c5bf6
@nikrb
update fetch-mock and lock file
1617151
@nikrb
add user actions unit test
09da6f9
@nikrb
fix: profile prop-types
f4d2968
@nikrb
fix: user types import
53b0c14
@nikrb
add user reducer unit tests
3079b43 
refactor user reducer
@nikrb
refactor login actions
ae47685
@nikrb
add login actions unit tests
7b63c39
@nikrb
fix: remove duplicate search redux actions
1381244
@nikrb
fix: rename user reducer unit test file
0e6839d
@nikrb
rename userReducer import to user to match other reducers in combineR…
05c4cbc 
…educers
@nikrb
fix: card missing image url
eea26a0
@nikrb
fix: refactor image url creator
052209d
@nikrb
fix: reject with missing url instead of console error log
e338bba
@nikrb
Merge pull request #98 from chingu-voyage3/fix/house-image-delete
b9187b1 
Fix/house image delete
@nikrb
Merge pull request #95 from chingu-voyage3/add-user-unit-test
08a68c3 
Add user unit test
@Mortuie
Started refactoring
e1d1504
@Mortuie
Deleting unecessary files
5809875
@Mortuie
Refactoring the USER REDUX stuff
6cef2c0
@Mortuie
Refactoring the User Tests
e641b01
@Mortuie
Removing an unecessary line
848daa8
@Mortuie
Added an index.js file
5cdcb73
@Mortuie
Fixed a bug on the auto complete scrolling
05b4de6
@Mortuie
Added minor changes
2cdf779
@Mortuie
Pushing refactor of the actions, but have now broken tests
b3da5c2
@Mortuie
Removed the cross fetch nonsense
a60a42c
@Mortuie
Moved all the tests back after reading an article
b126b7e
@Mortuie
Fixing the code I broke...
3519f95
@Mortuie
Fixed the styling of the register box
2382039
@Mortuie
Removed the stuff for pre and post pull/pushing or well just commente…
bd6c08b 
…d it out
nikrb
nikrb approved these changes Oct 14, 2018
View reviewed changes
Copy link
Contributor

@nikrb nikrb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor syntax issues (which can be fixed later). looks good otherwise.

vivekimsit and others added 24 commits October 14, 2018 22:17
@vivekimsit
WIP.
d0dae2f
@vivekimsit
WIP.
f1a65f0
@vivekimsit
Fix auth route and login flow.
96fdb20
@vivekimsit
Fix logout logic.
ed03b37
@vivekimsit
More fixes.
c022d7a
@vivekimsit
Fix login.
e3c622b
@vivekimsit
Revert to Mongo for now.
7538de7
@Mortuie
Initial commit for the prettier config
48d9619
@Mortuie
Prettier running on all files
9e31c89
@Mortuie
Adding the last files...
05086ba
@Mortuie
Fixing last linting issues and removing the unecessary shell script
6ee3b0a
@Mortuie
Merge pull request #123 from chingu-voyage3/feature/prettier
e793369 
Feature/prettier
@Mortuie
Initial commit
0d9c0ea
@Mortuie
Added hook
034b8f8
@Mortuie
Ran Prettier
579ffc7
@Mortuie
Revert "Feature/prettier"
707e37e
@vivekimsit
Merge pull request #126 from chingu-voyage3/revert-123-feature/prettier
bdee79a 
Revert "Feature/prettier"
@vivekimsit
Merge pull request #125 from chingu-voyage3/fixes/prettier
716eaea 
Prettier onto viveks branch.
@vivekimsit
Address comments.
3740588
@vivekimsit
Lint.
606d475
@vivekimsit
Merge pull request #122 from chingu-voyage3/fixes/error-handling
9727cc2 
Fixes/error handling
@Mortuie
Edited .gitignore
c322860
@Mortuie
Adding the using of the new Url parser
350b852
@Mortuie
Merge pull request #127 from chingu-voyage3/fix/removeDeprecationWarning
741d32a 
Removing mongo deprecation warning.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nikrb nikrb nikrb approved these changes

@vivekimsit vivekimsit vivekimsit approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Mortuie @vivekimsit @nikrb