Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Some more fixes for the chat page #2012

Merged
merged 10 commits into from
Aug 13, 2020
Merged

Some more fixes for the chat page #2012

merged 10 commits into from
Aug 13, 2020

Conversation

dngray
Copy link
Collaborator

@dngray dngray commented Aug 5, 2020

@dngray dngray requested a review from a team as a code owner August 5, 2020 04:39
pages/services/chat.html Outdated Show resolved Hide resolved
Co-authored-by: Mikaela Suomalainen <mikaela+git@mikaela.info>
Mikaela
Mikaela previously approved these changes Aug 5, 2020
@freddy-m

This comment has been minimized.

@dngray
Copy link
Collaborator Author

dngray commented Aug 8, 2020

We should probably look at #2015 as well for this issue.

@dngray dngray changed the title Fix fragmented sentence on chat page. Some more fixes for the chat page Aug 8, 2020
@dngray dngray added the WIP active work in progress, do not merge or PR (yet)! label Aug 8, 2020
pages/services/chat.html Outdated Show resolved Hide resolved
@dngray dngray merged commit 8d398ce into master Aug 13, 2020
@dngray dngray deleted the pr-fix_fragment_chat_page branch August 13, 2020 13:38
@dngray
Copy link
Collaborator Author

dngray commented Aug 25, 2020

It seems this didn't take place on the live page https://www.privacytools.io/services/chat @jonaharagon.

Two things I noticed that didn't seem to make it was the CoC, and the refinement of the Cloudflare related question.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
WIP active work in progress, do not merge or PR (yet)!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants