commudle/commudle-ng

Add cookie consent message

Closed this issue ยท 8 comments

Simple, create a component for cookie consent.

Should be displayed once per session. Use local storage to remember not asking the user again.

In the next step we will link this to an API to store the consent per user (if we need to)

I would like to work on this one!

@jrstrayhorn thanks a lot. Let's do this, assigning this issue to you.
Would be great if you can just put a rough timeline we're looking at!

Hi @arpansac! I plan to start working on it today and have a PR in for it over next 24 to 48 hrs.

Going to take the approach that patreon.com uses for their cookie consent (see screen shot below), between top nav and main content for commudle site - that work for you?

patreon-cookie-consent

Sounds cool @jrstrayhorn
As for the position of the consent, we're using Nebular library

Probably using the window component might make it easier for you. Preferred position is any place at the bottom :)

Any updates @jrstrayhorn ?

Hi! @arpansac - I left you questions and updated comments on the pull request - #84 a few days ago ๐Ÿ˜Š

Oops, somehow didn't get a notification mail :P

Let me merge and test it, if everything's fine, it'll be live in the next 12 hours ๐Ÿ‘

I guess 1 change was pending, I was just figuring it out myself. Your code will be live in 1 hour :D