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.

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

Code review #3

Open
wants to merge 32 commits into
base: review2
Choose a base branch
from
Open

Code review #3

wants to merge 32 commits into from

Conversation

matiasgarcia91
Copy link
Collaborator

No description provided.

Copy link
Collaborator Author

@matiasgarcia91 matiasgarcia91 left a comment

Choose a reason for hiding this comment

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

Great work Jend! Really not much to say I see you are learning a lot of different CSS techniques and libraries, it might be a pain but it's good that you get to see it. Let's demo it tomorrow, I didn't run it!

export const USERPROFILE_SIGNED_UP = "userProfile/SIGNED_UP";
export const USERADDRESS_SIGNED_UP = "userAddress/SIGNED_UP";

const baseUrl = "http://localhost:5000";
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

with axios you can create an instance in another file where you set the baseUrl and then import axios from that file from anywhere, try it out!

// src/axios.js
import axios from "axios";

const instance = axios.create({
  baseURL: "http://localhost:4000"
});
export default instance;

Copy link
Owner

Choose a reason for hiding this comment

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

thanks! will try it out.

Copy link
Owner

Choose a reason for hiding this comment

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

got this one to work. very cool suggestion. thanks!

userProfileData
);

dispatch(userProfileSignUp_Sucess(responseData.data));
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

if you are using camelCase for function and variables (which is the way to go) don't use underscores _

Copy link
Owner

Choose a reason for hiding this comment

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

thanks for the tip. will update and have it implemented by the next push.

<h1>Sasha & John Digweed x World Fashion Centre Amsterdam</h1>
<p>
In an ever-evolving electronic music landscape, Sasha and John
Digweed have managed to stay as relevant as ever. After
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

you into him? he's good

Copy link
Owner

Choose a reason for hiding this comment

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

yep agree. hahaha

@jendhordejan jendhordejan temporarily deployed to book-a-cook March 29, 2020 14:40 Inactive
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