Skip to content

Initial structure#2

Merged
rajikaimal merged 9 commits intocloudlibz:masterfrom
amrita019:user-auth
Jul 3, 2019
Merged

Initial structure#2
rajikaimal merged 9 commits intocloudlibz:masterfrom
amrita019:user-auth

Conversation

@amrita019
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown
Member

@rajikaimal rajikaimal left a comment

Choose a reason for hiding this comment

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

Format all code with Prettier

Comment thread node-backend/server.js Outdated
@@ -0,0 +1,19 @@
var express = require("express");
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's use const and let for all variable declarations.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Okay

Comment thread node-backend/helpers/utils.js Outdated
require('dotenv').config();


module.exports = {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Format code with Prettier

Copy link
Copy Markdown
Member

@rajikaimal rajikaimal left a comment

Choose a reason for hiding this comment

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

Let's rename node-backend to server.

2. Send the selected service provider from front end
3. node-backend renamed to server
Copy link
Copy Markdown
Member

@rajikaimal rajikaimal left a comment

Choose a reason for hiding this comment

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

Let's rename react-frontend to client

@rajikaimal
Copy link
Copy Markdown
Member

@amrita019 Please rename react-frontend to client.

amrita019 added 2 commits July 3, 2019 02:25
2. Design Navigation Bar in dashboard
3. Add an option to create service in nav bar which inflates modal in return
4. WIP: Create API endpoint for create service
Comment thread client/src/_services/user.services.js Outdated
body: JSON.stringify(user)
};

return fetch(`http://localhost:4000/signup`, requestOptions).then(
Copy link
Copy Markdown
Member

@rajikaimal rajikaimal Jul 3, 2019

Choose a reason for hiding this comment

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

Let's move URLs to a different file.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Sure

Comment thread server/routes/auth.js Outdated
@@ -0,0 +1,26 @@
let express = require("express");
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Change these variables to const

@rajikaimal rajikaimal changed the title User authentication Initial structure Jul 3, 2019
2. Changed occurences of let to const
@rajikaimal rajikaimal merged commit 3fdd9ad into cloudlibz:master Jul 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants