HadiyahL · g1st · Dec 10, 2020 · Dec 9, 2020 · Dec 10, 2020 · Dec 10, 2020
diff --git a/client/src/components/index.js b/client/src/components/index.js
@@ -1,2 +1,3 @@
 export { default as CreateWorkerForm } from "./CreateWorkerForm";
+
 export { default as ShowWorkersForm } from "./ShowWorkersForm";
diff --git a/client/src/pages/index.js b/client/src/pages/index.js
@@ -1,2 +1,3 @@
 export { default as CreateWorker } from "./CreateWorker.js";
+
 export { default as ShowWorkers } from "./ShowWorkers.js";
diff --git a/client/src/service.js b/client/src/service.js
@@ -5,6 +5,13 @@ export const postWorkers = async (data) => {
 	return response.data;
 };
 
+
+
+
+
+
+
+
 export const getWorkers = async (data) => {
 	const response = await get("/workers", data);
 	return response.data;
Original file line number	Diff line number	Diff line change
		@@ -1,2 +1,3 @@
		export { default as CreateWorkerForm } from "./CreateWorkerForm";

Copy link Collaborator Harriethw Dec 10, 2020 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. Please could you explain why you've added this commit? Thanks to version control there isn't any need to add white space - if you absolutely need to add a TODO you can add commented out lines. Copy link Collaborator Author GTabala Dec 10, 2020 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. I just fixed the code according to all the comments from the reviews. Copy link Collaborator g1st Dec 10, 2020 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. @Harriethw I originally added that file to make it more clear and tidy when we will import them inside `App.js` or `/pages`. More info here https://www.digitalocean.com/community/tutorials/react-index-js-public-interfaces#using-indexjs Copy link Collaborator Harriethw Dec 10, 2020 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. @g1st ah I just meant that we don't need empty white space, which is all this commit seems to add unless I'm missing something? Copy link Collaborator Author GTabala Dec 10, 2020 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. Yesterday we made a customer list with Hadiyah. It doesn't work on her computer. I made the same code for workers and it works. I added empty spaces for her code. :) Copy link Collaborator g1st Dec 10, 2020 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. Oh sorry @Harriethw yes I agree. I already proposed @GTabala to use Prettier and it can take care of those empty spaces for us! Copy link Collaborator Author GTabala Dec 10, 2020 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. Prettier doesn't delete empty lines. Copy link Collaborator g1st Dec 10, 2020 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. Hmm, I don't know who does it then but when I save a file with loads of empty lines they get deleted for me.
		export { default as ShowWorkersForm } from "./ShowWorkersForm";
Original file line number	Diff line number	Diff line change
		@@ -1,2 +1,3 @@
		export { default as CreateWorker } from "./CreateWorker.js";

		export { default as ShowWorkers } from "./ShowWorkers.js";