Skip to content

/ ghostery-extension

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

Jump to bottom

GH-2035: Update In-App Plus/Premium pricing #596

Merged
merged 5 commits into from Aug 17, 2020
+16 −12
Merged

GH-2035: Update In-App Plus/Premium pricing #596

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
cc7679a
Update in app plus/premium pricing
benstrumeyer Aug 12, 2020
716af8a
Revert to 7-day free trial string
benstrumeyer Aug 13, 2020
28071d2
Fix linter
benstrumeyer Aug 13, 2020
5f30a23
Update snapshot
benstrumeyer Aug 13, 2020
bef6eb1
Merge branch 'develop' into GH-2035
christophertino Aug 17, 2020
File filter
Filter file types
Clear filters
Jump to
Jump to file
Failed to load files.

Always

Just for now

Prev Next
Loading status checks…

Update snapshot

  • Loading branch information
@benstrumeyer
benstrumeyer committed Aug 13, 2020
commit 5f30a238feba8acc2a2e83abdf977b3be5e7c4fd
18 app/hub/Views/PlusView/__tests__/__snapshots__/PlusView.test.jsx.snap
View file
Open in desktop
@@ -97,9 +97,12 @@ exports[`app/hub/Views/PlusView component Snapshot tests with react-test-rendere
>
<div
className="PlusView__manifestoText columns small-12 medium-10 large-8 text-center"
>
hub_supporter_manifesto
</div>
dangerouslySetInnerHTML={
Object {
"__html": "hub_supporter_manifesto",
}
This conversation was marked as resolved by wlycdgr
Comment on lines +101 to +103

This comment has been minimized.

Sign in to view
@wlycdgr

wlycdgr Aug 17, 2020
Member

how come you're using this Object syntax here vs dangerouslySetInnerHTML={{ __html:...?

This comment has been minimized.

Sign in to view
@wlycdgr

wlycdgr Aug 17, 2020
Member

Never mind, I missed that these are in snap files!
}
/>
</div>
</div>
<div
@@ -426,9 +429,12 @@ exports[`app/hub/Views/PlusView component Snapshot tests with react-test-rendere
>
<div
className="PlusView__manifestoText columns small-12 medium-10 large-8 text-center"
>
hub_supporter_manifesto
</div>
dangerouslySetInnerHTML={
Object {
"__html": "hub_supporter_manifesto",
}
This conversation was marked as resolved by wlycdgr
Comment on lines +433 to +435

This comment has been minimized.

Sign in to view
@wlycdgr

wlycdgr Aug 17, 2020
Member

Same question re: the Object syntax. No complaints I just haven't seen it done this way, so, curious.

This comment has been minimized.

Sign in to view
@wlycdgr

wlycdgr Aug 17, 2020
Member

Never mind, I missed that these are in snap files!
}
/>
</div>
</div>
<div
ProTip! Use n and p to navigate between commits in a pull request.