Skip to content
This repository has been archived by the owner on Nov 10, 2023. It is now read-only.

(Feature) Implement GDPR requirements #490

Merged
merged 8 commits into from
Jan 31, 2020

Conversation

gabitoesmiapodo
Copy link
Contributor

Closes #427

  • Removed X button from cookies banner.
  • Added Accept preferences link to cookies banner.
  • Added footer.
  • Added footer links as requested.
  • Removed redundant links from sidebar.

It should look something like these:

Screen Shot 2020-01-29 at 19 42 56
Screen Shot 2020-01-29 at 19 43 35
Screen Shot 2020-01-29 at 19 42 43
Screen Shot 2020-01-29 at 19 51 48

@ghost
Copy link

ghost commented Jan 29, 2020

Travis automatic deployment:
https://pr490--safereact.review.gnosisdev.com

mmv08
mmv08 previously requested changes Jan 30, 2020
package.json Outdated Show resolved Hide resolved
src/components/Sidebar/index.jsx Outdated Show resolved Hide resolved
@ghost
Copy link

ghost commented Jan 30, 2020

Travis automatic deployment:
https://pr490--safereact.review.gnosisdev.com

@ghost
Copy link

ghost commented Jan 30, 2020

Travis automatic deployment:
https://pr490--safereact.review.gnosisdev.com

@ghost
Copy link

ghost commented Jan 30, 2020

Travis automatic deployment:
https://pr490--safereact.review.gnosisdev.com

* development:
  (fix) Notifications' visual inconsistencies (#492)

# Conflicts:
#	src/components/layout/PageFrame/index.jsx
@ghost
Copy link

ghost commented Jan 31, 2020

Travis automatic deployment:
https://pr490--safereact.review.gnosisdev.com

@gabitoesmiapodo gabitoesmiapodo dismissed mmv08’s stale review January 31, 2020 14:05

The issue seems not to be critical (script running on Windows)

@gabitoesmiapodo gabitoesmiapodo merged commit 84926fd into development Jan 31, 2020
@gabitoesmiapodo gabitoesmiapodo deleted the feature/#427-implement-GDPR-requirements branch January 31, 2020 14:06
@fernandomg fernandomg mentioned this pull request Feb 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants