Skip to content

Commit 09df267

Browse files
committed
fix unbook event + fix imports
1 parent 6363ca4 commit 09df267

20 files changed

Lines changed: 37 additions & 38 deletions

src/bookmark/BookmarksSidebar.js

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -12,9 +12,15 @@ import { trackLinkUnBookmark, Attributes } from 'src/lib/analytics'
1212

1313
const BookmarkItem = ({ item, appendRef = true }) => {
1414
const { dispatcher } = useContext(PreferencesContext)
15+
const analyticsAttrs = {
16+
[Attributes.TRIGERED_FROM]: 'bookmarks',
17+
[Attributes.TITLE]: item.title,
18+
[Attributes.LINK]: item.url,
19+
[Attributes.SOURCE]: item.source,
20+
}
1521
const unBookmark = () => {
1622
dispatcher({ type: 'unBookmarkItem', value: item })
17-
trackLinkUnBookmark(item.source)
23+
trackLinkUnBookmark(analyticsAttrs)
1824
}
1925
return (
2026
<MenuItem
@@ -26,12 +32,10 @@ const BookmarkItem = ({ item, appendRef = true }) => {
2632
<CardLink
2733
link={item.url}
2834
appendRef={appendRef}
29-
analyticsAttributes={{
30-
[Attributes.TRIGERED_FROM]: 'bookmarks',
31-
[Attributes.TITLE]: item.title,
32-
[Attributes.LINK]: item.url,
33-
[Attributes.SOURCE]: item.source,
34-
}}>{`${item.title}`}</CardLink>
35+
analyticsAttributes={analyticsAttrs}
36+
>
37+
{`${item.title}`}
38+
</CardLink>
3539
</MenuItem>
3640
)
3741
}

src/cards/DevToCard.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ import { AiOutlineLike } from 'react-icons/ai'
1111
import CardItemWithActions from '../components/CardItemWithActions'
1212
import ColoredLanguagesBadge from '../components/ColoredLanguagesBadge'
1313
import SelectableCard from '../components/SelectableCard'
14-
import { GLOBAL_TAG, MY_LANGUAGES_TAG, MAX_MERGED_ITEMS_PER_LANGUAGE } from '../constants'
14+
import { GLOBAL_TAG, MY_LANGUAGES_TAG, MAX_MERGED_ITEMS_PER_LANGUAGE } from '../Constants'
1515
import { mergeMultipleDataSources } from '../utils/DataUtils'
1616
import { trackCardLanguageSelect } from 'src/lib/analytics'
1717
import { Attributes } from 'src/lib/analytics'

src/cards/HashNodeCard.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ import { AiTwotoneHeart } from 'react-icons/ai'
1111
import CardItemWithActions from '../components/CardItemWithActions'
1212
import ColoredLanguagesBadge from '../components/ColoredLanguagesBadge'
1313
import SelectableCard from '../components/SelectableCard'
14-
import { GLOBAL_TAG, MY_LANGUAGES_TAG, MAX_MERGED_ITEMS_PER_LANGUAGE } from '../constants'
14+
import { GLOBAL_TAG, MY_LANGUAGES_TAG, MAX_MERGED_ITEMS_PER_LANGUAGE } from '../Constants'
1515
import { mergeMultipleDataSources } from '../utils/DataUtils'
1616
import { trackCardLanguageSelect, Attributes } from 'src/lib/analytics'
1717

src/cards/MediumCard.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ import { MdAccessTime } from 'react-icons/md'
1010
import { MdWavingHand } from 'react-icons/md'
1111
import CardItemWithActions from '../components/CardItemWithActions'
1212
import SelectableCard from '../components/SelectableCard'
13-
import { GLOBAL_TAG, MY_LANGUAGES_TAG, MAX_MERGED_ITEMS_PER_LANGUAGE } from '../constants'
13+
import { GLOBAL_TAG, MY_LANGUAGES_TAG, MAX_MERGED_ITEMS_PER_LANGUAGE } from '../Constants'
1414
import { mergeMultipleDataSources } from '../utils/DataUtils'
1515
import { trackCardLanguageSelect } from 'src/lib/analytics'
1616
import { Attributes } from 'src/lib/analytics'

src/cards/ReposCard.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import PreferencesContext from '../preferences/PreferencesContext'
77
import CardLink from '../components/CardLink'
88
import CardItemWithActions from '../components/CardItemWithActions'
99
import SelectableCard from '../components/SelectableCard'
10-
import { GLOBAL_TAG, MY_LANGUAGES_TAG, MAX_MERGED_ITEMS_PER_LANGUAGE } from '../constants'
10+
import { GLOBAL_TAG, MY_LANGUAGES_TAG, MAX_MERGED_ITEMS_PER_LANGUAGE } from '../Constants'
1111
import { mergeMultipleDataSources } from '../utils/DataUtils'
1212
import ColoredLanguagesBadge from '../components/ColoredLanguagesBadge'
1313
import { trackCardLanguageSelect, trackCardDateRangeSelect, Attributes } from 'src/lib/analytics'

src/components/AppContentLayout.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import React, { useContext, useState } from 'react'
2-
import { SUPPORTED_CARDS } from '../constants'
2+
import { SUPPORTED_CARDS } from '../Constants'
33
import PreferencesContext from '../preferences/PreferencesContext'
44
import BottomNavigation from './BottomNavigation'
55
import { isDesktop } from 'react-device-detect'

src/components/BottomNavigation.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import React, { useContext } from 'react'
2-
import { SUPPORTED_CARDS } from '../constants'
2+
import { SUPPORTED_CARDS } from '../Constants'
33
import PreferencesContext from '../preferences/PreferencesContext'
44
import { AiOutlineMenu } from 'react-icons/ai'
55

src/components/CardComponent.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import React, { useContext } from 'react';
22
import { BsBoxArrowInUpRight } from 'react-icons/bs';
3-
import { APP } from '../constants'
3+
import { APP } from '../Constants'
44
import PreferencesContext from '../preferences/PreferencesContext'
55

66
function CardComponent({ icon, title, children, fullBlock, link }) {

src/components/CardItemWithActions.js

Lines changed: 8 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -19,21 +19,16 @@ export default function CardItemWithActions({ cardItem, item, index, source }) {
1919
},
2020
})
2121
setIsBookmarked(!isBookmarked)
22-
22+
const analyticsAttrs = {
23+
[Attributes.TRIGERED_FROM]: 'card',
24+
[Attributes.TITLE]: item.title,
25+
[Attributes.LINK]: item.url,
26+
[Attributes.SOURCE]: source,
27+
}
2328
if (isBookmarked) {
24-
trackLinkUnBookmark({
25-
[Attributes.TRIGERED_FROM]: 'bookmarks',
26-
[Attributes.TITLE]: item.title,
27-
[Attributes.LINK]: item.url,
28-
[Attributes.SOURCE]: source,
29-
})
29+
trackLinkUnBookmark(analyticsAttrs)
3030
} else {
31-
trackLinkBookmark({
32-
[Attributes.TRIGERED_FROM]: 'bookmarks',
33-
[Attributes.TITLE]: item.title,
34-
[Attributes.LINK]: item.url,
35-
[Attributes.SOURCE]: source,
36-
})
31+
trackLinkBookmark(analyticsAttrs)
3732
}
3833
}
3934
useEffect(() => {

src/components/Changelog.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import ReactMarkdown from 'react-markdown'
55
import axios from 'axios'
66
import { format } from 'timeago.js'
77
import BeatLoader from 'react-spinners/BeatLoader'
8-
import { APP } from '../constants'
8+
import { APP } from '../Constants'
99
import PreferencesContext from '../preferences/PreferencesContext'
1010

1111
function Changelog({}) {

0 commit comments

Comments
 (0)