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

Show cookie banner only on daily.co screen #516

Open
HugoGresse opened this issue Jan 2, 2021 · 12 comments
Open

Show cookie banner only on daily.co screen #516

HugoGresse opened this issue Jan 2, 2021 · 12 comments
Labels
high priority webApp Web app related issues

Comments

@HugoGresse
Copy link
Member

No description provided.

@HugoGresse HugoGresse added webApp Web app related issues high priority labels Jan 2, 2021
@MattiaPrimavera
Copy link
Collaborator

wasn't this needed by daily for the call to work ?

@HugoGresse
Copy link
Member Author

at least for the prenium call so.

@MattiaPrimavera MattiaPrimavera assigned ghost Jan 6, 2021
@ghost
Copy link

ghost commented Jan 7, 2021

Could you give me more details on which screen I have to remove this banner please ?

@HugoGresse
Copy link
Member Author

/premium-call/*
on android & ios APPS (cordova/ionic)

@MattiaPrimavera
Copy link
Collaborator

What did we need the cookies for ? If only for daily.co calls, we may move the banner display inside daily pre-join screens ? @HugoGresse @ntchou

@HugoGresse
Copy link
Member Author

and GA maybe?

@MattiaPrimavera MattiaPrimavera changed the title Remove cookie banner on mobile native apps Show cookie banner only on daily.co screen Jan 7, 2021
@MattiaPrimavera
Copy link
Collaborator

Separate issue for GA

@MattiaPrimavera
Copy link
Collaborator

I think we can change this issue: in the wireframes there is a specific page for accepting cookies, we'd rather directly move the cookies demand inside that page

@MattiaPrimavera
Copy link
Collaborator

Do you agree @HugoGresse @ntchou ? In that case we change the issue title or check if there's already an existing one for this

@HugoGresse
Copy link
Member Author

having a separate page for cookie is hard to do, at least regarding the law and the solution (cookieBot need to be payed for a custom design). By the way, we can probably move to https://www.axeptio.eu/fr/home in the future.

@MattiaPrimavera
Copy link
Collaborator

I meant that we could show the same UI for accepting cookies, but only show it when clicking on the button in page 5 - accepter les cookies (see the wireframes). Is that hard to do ? Does the law force to have the banner appear in the homepage, even if we only use/store cookies in another page ?

@HugoGresse
Copy link
Member Author

we can't right now due to Crisp using those cookie, but maybe we could yes, we need to ensure that we are not using cookie for anything else than Daily after Crisp is replaced by landbot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high priority webApp Web app related issues
Projects
None yet
Development

No branches or pull requests

2 participants