Skip to content

Commit

Permalink
Update readme and fix linter issue
Browse files Browse the repository at this point in the history
  • Loading branch information
gretahayes19 committed Mar 2, 2023
1 parent 1ac0dd5 commit b76463d
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 5 deletions.
1 change: 0 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ lib

# testing
/coverage
.jest

# production
/build
Expand Down
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -28,9 +28,9 @@ The Design System tech lead and Product Designers are the main decision makers &

### Set up local variables

After cloning the repo, create `.npmrc` and `.env` files in the root level directory, as well as a `.jest` directory with a file named `setEnvVars.js` inside. These gitignored files will store our auth tokens and other local variables.
After cloning the repo, create `.npmrc` and `.env` files in the root level directory. These gitignored files will store our auth tokens and other local variables.

Copy the respective contents of into each new file from the `Design System local variables` note located in our password manager (1Password).
Copy the respective contents into each new file from the `Design System local variables` note located in our password manager (1Password).

### `yarn install`

Expand Down
4 changes: 2 additions & 2 deletions src/hoc/withTrackedClick.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,13 @@ export default function withTrackedClick(Target) {

const { eventData, event, ...rest } = props;

function handleClick(clickEvent) {
const handleClick = (clickEvent) => {
props.tracking.trackEvent({ ...eventData, event });

if (props.onClick) {
props.onClick(clickEvent);
}
}
};

return <Target {...rest} onClick={handleClick} />;
};
Expand Down

0 comments on commit b76463d

Please sign in to comment.