[ENHANCEMENT] Move "Accept Cookie" to Storage

[elisei]

Description

The "acceptance of cookies" is currently in session, which creates unwanted behavior, customer returns to the site and even being logged in, or even with persistence of their cart, needs to accept cookies again.

Priority of the change:

• High
• Medium
• Low

Which problem the feature would fix?
O comportamento para dos cookies

Describe the solution as simple as possible
Move the cookie to local storage.

Additional context

Contributing

• Would you want to contribute and to be assigned to this enhancement?

[thomas-kl1]

Hi @elisei

This comment might interest you: #9 (comment)
There's already a similar feature in Magento so we don't need the one provided by this module (as it only shows a disclosure and don't allow to consent to and configure stored cookies)

[elisei]

Hi @thomas-kl1

I believe that the topic you mention is not quite what I propose, maybe the pull clarifies it better:
elisei@f325aff

The main proposed changes are:

• Set visible to false. It will only be displayed if there is no cookie.
• Instead of setting the cookie to "session" I move it to local storage. This will make the once-dated authorization remain as authorized until the data is cleared. With this, the client returns to the site, after a period longer than the session, he is still authorized.

This last point aims to correct situations such as:

• Customer comes back to the site, the session time for account cookies has not yet expired, but we still ask for his authorization to record cookies! o.O Hours if I have not yet authorized cookies how am I still logged in?

[elisei]

PS. In Pull also add a check so that the popup is only visible if it is not a bot or lighthouse, thus avoiding loss in the CLS factor.

[thomas-kl1]

@elisei feel free to open a pull request