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

7386 - Content Update #7590

Merged
merged 6 commits into from
Oct 13, 2021
Merged

7386 - Content Update #7590

merged 6 commits into from
Oct 13, 2021

Conversation

fessehaye
Copy link
Contributor

@fessehaye fessehaye commented Oct 8, 2021

@mofodevops mofodevops temporarily deployed to foundation-s-7386-updat-lt7w6l October 8, 2021 23:39 Inactive
Copy link
Collaborator

@danielfmiranda danielfmiranda left a comment

Choose a reason for hiding this comment

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

Hey SImon! Nothing seems to be appearing underneath the AI section:
https://foundation-s-7386-updat-lt7w6l.herokuapp.com/en/privacynotincluded/fill-what-least-page-low/
image

@mofodevops mofodevops temporarily deployed to foundation-s-7386-updat-lt7w6l October 9, 2021 00:07 Inactive
Copy link

@sabrinang sabrinang left a comment

Choose a reason for hiding this comment

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

  • points in pink are missing
  • i icon ares missing from sections

image

I'm referencing the both spreadsheet and figma file

…uct_page.html

Co-authored-by: Théo Chevalier <theo.chevalier11@gmail.com>
@mofodevops mofodevops temporarily deployed to foundation-s-7386-updat-lt7w6l October 12, 2021 17:31 Inactive
Copy link
Contributor

@TheoChevalier TheoChevalier left a comment

Choose a reason for hiding this comment

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

Thanks Simon! Looks good

@sabrinang
Copy link

Note:

  • "Biometric" has changed to "Body related" — see line 29 on spreadheet
  • AI section might have adjustments after legal meeting next Friday

@mofodevops mofodevops temporarily deployed to foundation-s-7386-updat-lt7w6l October 13, 2021 22:35 Inactive
Copy link

@sabrinang sabrinang left a comment

Choose a reason for hiding this comment

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

👍 Met with Simon to walk through content and reviewed

Copy link
Collaborator

@danielfmiranda danielfmiranda left a comment

Choose a reason for hiding this comment

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

Hey Simon, just gave this another review and everything looks good to me. Thanks!

@mofodevops mofodevops temporarily deployed to foundation-s-7386-updat-lt7w6l October 13, 2021 23:13 Inactive
@mofodevops mofodevops temporarily deployed to foundation-s-7386-updat-lt7w6l October 13, 2021 23:35 Inactive
@fessehaye fessehaye merged commit 8213b83 into pni-q3-2021 Oct 13, 2021
@fessehaye fessehaye deleted the 7386-updates branch October 13, 2021 23:45
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.

None yet

5 participants