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

316 more cookies #335

Merged
merged 3 commits into from
Dec 7, 2023
Merged

316 more cookies #335

merged 3 commits into from
Dec 7, 2023

Conversation

alishaevn
Copy link
Contributor

@alishaevn alishaevn commented Dec 6, 2023

Story

update the cookie policy to reflect webstore cookies

ref: https://assaydepot.slack.com/archives/C05U031L0V9/p1701809599298789?thread_ts=1701363833.246969&cid=C05U031L0V9

Expected Behavior Before Changes

Expected Behavior After Changes

  • the cookie policy wording has been updated to account for info that's specific to the webstore
  • the option to enable/disable cookies is hidden altogether since we only have essential cookies right now

Screenshots / Video

to see the full coolie policy, refer to "/legal-notices/cookie-policy" on the vercel preview site below!

image

Copy link

vercel bot commented Dec 6, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
webstore-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 7, 2023 2:58pm

@alishaevn alishaevn linked an issue Dec 6, 2023 that may be closed by this pull request
8 tasks
@@ -31,9 +31,9 @@ const cookiePolicy = (
<ul>
<li><strong>Strictly necessary cookies.</strong> These are cookies that are required for the operation of our site. They include, for example, cookies that enable you to log into secure areas of our site or make use of services. We do not require your consent to place these cookies. Nevertheless, you may be able to block these cookies yourself on your device/browser, but restricting these cookies is likely to mean that our site will not work as you would expect and certain functionality may be inoperable.</li>
<p style={{textDecoration: 'underline'}}><strong>Non-essential cookies</strong></p>
<li><strong>Analytical/performance cookies.</strong> They allow us to recognise and count the number of visitors and to see how visitors move around our site when they are using it. This helps us to improve the way our site works, for example, by ensuring that users are finding what they are looking for easily. We use cookies to compile visitor statistics such as how many people have visited our site, how they reached our site, what type of technology they are using (e.g. Mac or Windows which helps to identify when our site isn't working as it should for particular technologies), how long they spend on the site, what page they look at etc. This helps us to continuously improve our website.</li>
{/* <li><strong>Analytical/performance cookies.</strong> They allow us to recognise and count the number of visitors and to see how visitors move around our site when they are using it. This helps us to improve the way our site works, for example, by ensuring that users are finding what they are looking for easily. We use cookies to compile visitor statistics such as how many people have visited our site, how they reached our site, what type of technology they are using (e.g. Mac or Windows which helps to identify when our site isn't working as it should for particular technologies), how long they spend on the site, what page they look at etc. This helps us to continuously improve our website.</li>
Copy link
Contributor Author

@alishaevn alishaevn Dec 6, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we aren't using analytical, functional or targeting cookies on the webstore. at least not in the ways described.

@alishaevn alishaevn merged commit 79ea124 into main Dec 7, 2023
4 checks passed
@alishaevn alishaevn deleted the 316-more-cookies branch December 7, 2023 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Scientist.com Cookie Policy
2 participants