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

The series of updates in this PR will solve dependency installation and project build issues #93

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
fa105bb
- install the dependencies and update by using "yarn upgrade-interact…
arsahindev Jan 9, 2024
395c127
solve "JSX props should not use functions react/jsx-no-bind" for Edi…
arsahindev Jan 9, 2024
35f39df
solve "JSX props should not use functions react/jsx-no-bind" for Nei…
arsahindev Jan 9, 2024
67f5030
solve "JSX props should not use functions react/jsx-no-bind" for Nei…
arsahindev Jan 9, 2024
4ef762e
solve "JSX props should not use functions react/jsx-no-bind" for Not…
arsahindev Jan 9, 2024
93b044e
solve "JSX props should not use functions react/jsx-no-bind" for Sig…
arsahindev Jan 9, 2024
4ef796f
solve "JSX props should not use functions react/jsx-no-bind" for Sig…
arsahindev Jan 9, 2024
dd70237
solve "A control must be associated with a text label jsx-a11y/contr…
arsahindev Jan 9, 2024
302c509
solve "A control must be associated with a text label jsx-a11y/contr…
arsahindev Jan 9, 2024
993caef
solve "A control must be associated with a text label jsx-a11y/contr…
arsahindev Jan 9, 2024
9457194
I've fixed the proglem of me unintentionally writing useSelector inst…
arsahindev Jan 9, 2024
37acad8
update snapshots of component snapshot tests
arsahindev Jan 9, 2024
4c0abc5
Merge pull request #3 from ARS-coding/1-update-to-the-dependencies-to…
arsahindev Jan 9, 2024
bf3240e
change the react-hook-fdrm version from '^7.14.2' to '7.45.4'
arsahindev Jan 9, 2024
8d89a4c
Merge pull request #4 from ARS-coding/1-update-to-the-dependencies-to…
arsahindev Jan 9, 2024
8a2503f
add google analytics 4 tag
arsahindev Apr 6, 2024
0c8adce
Merge pull request #5 from ARS-coding/1-update-to-the-dependencies-to…
arsahindev Apr 6, 2024
da5bf98
add .nvmrc
arsahindev Apr 6, 2024
d363acb
pass NODE_OPTIONS='--openssl-legacy-provider' env variable to start a…
arsahindev Apr 6, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
solve "JSX props should not use functions react/jsx-no-bind" for Neig…
…hborSummaryModal
  • Loading branch information
arsahindev committed Jan 9, 2024
commit 67f5030fdc82e6080fcc32001c00d49984c857b1
19 changes: 13 additions & 6 deletions src/components/NeighborSummaryModal/index.jsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import React from "react";
import React, { useCallback } from "react";

import { useTranslation } from "react-i18next";

Expand Down Expand Up @@ -37,7 +37,7 @@ function NeighborSummaryModal({

const { t } = useTranslation();

function sendEmail() {
const sendEmail = useCallback(() => {
setEmailAlertStatus("empty");
send("service_9rwjsp6", "template_qlu5ttf", {
from_name: senderFullName,
Expand All @@ -47,9 +47,16 @@ function NeighborSummaryModal({
})
.then(() => setEmailAlertStatus("success"))
.catch(() => setEmailAlertStatus("danger"));
}

function handleInvitation() {
}, [
email,
firstName,
lastName,
senderEmail,
senderFullName,
setEmailAlertStatus,
]);

const handleInvitation = useCallback(() => {
dispatch({ type: "neighborSummary" });
sendEmail();
firestore
Expand All @@ -76,7 +83,7 @@ function NeighborSummaryModal({
id: uuidv4(),
}),
});
}
}, [dispatch, email, firstName, lastName, sendEmail, senderFullName, uid]);

return (
Object.keys(selectedNeighbor).length !== 0 && (
Expand Down