Refactored InfoModal.jsx and LoginModal.jsx

[tentsundue]

InfoModal.jsx

Callback Functions Added

• handleClose replaced previous callback function for onClose prop in Modal component
• handleShowEmail replaced previous callback function for onClick prop in ViewContactButton
• handleShare replaced previous callback function for onClick prop in the Share Button component

Ternary Operators Moved and Their Variables

• ownerTag represents the blue owner tag for a given item
• lostTag represents the red lost tag for a given item
• foundTag represents the green found tag for a given item
• lostDateText represents the date at which a given item was lost
• foundDateText represents the date at which a given item was found
• viewContactButton represents the blue "View Contact" button
• showContactButton represents the button shown after clicking the "View Contact" button, contains contact info

Axios.Delete Moved

I was able to move axios.delete from handleDelete into utils/ApiUtils.js

LoginModal.jsx

Callback Functions Added

• handleSignInGoogle replaced previous callback function for onClick prop in the Sign In Button component

Ternary Operators Moved and Their Variables

• signInErrorAlert represents the red "Can't sign in?" Alert box
• welcomeMessage represents the welcome text shown on the interface

Small Note

For LoginModal.jsx, the handleSignInGoogle function had to utilize signInGoogle within it's dependency list, otherwise there could be a risk of performance issues in the future. However, in order for that to happen, I had to wrap signInGoogle in its own useCallback() hook and commented out the original function for now.

[NwinNwin]

Yoooo Tenzin! Thank you, but I did notice something that would be very helpful in making this file more readable. Could you please add comments to all the functions/variables similar to the Map.jsx?

[tentsundue]

Hey Nwin, my fault for not adding some earlier but I just pushed new changes with comments for each function and variable I added. Let me know if you need them to be more descriptive, technical, etc.