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

Create chat_effects.md #191

Merged
merged 5 commits into from
Apr 11, 2022
Merged

Conversation

daniellekirkwood
Copy link
Contributor

@daniellekirkwood daniellekirkwood commented Mar 2, 2022

Fixes #190

We discussed in the Product Team internal the need for clarity on Chat Effects. As this is product strategy that impacts the community and those wishing to contribute to our codebase, it makes sense to create documentation in Element-Meta where we can track, approve, and update any strategy or product decisions such as this...

New document here: https://github.com/vector-im/element-meta/blob/daniellekirkwood-patch-190/docs/chat_effects.md

Also added a docs/ folder to Element-Meta repo.

Add info on Mobile (Android and iOS) capabilities.
Updated directions to turn on chat effects
@daniellekirkwood
Copy link
Contributor Author

@nadonomy & @kittykat there's 2 reviews here...

  1. These are my thoughts on Chat Effects and the stage we're in with them right now. What are your thoughts on this? Anything you'd change?
  2. How do you feel about keeping community relevant strategy and product documentation stored this way? I'm personally enjoying the "GitHub is a source of truth" principle and using Markdown and a ./docs/ folder allows us to do a product review before publish (unlike wiki pages) but keeps all our thinking in the open ready for community contribution against product values and ideas... Thoughts... :D

@daniellekirkwood daniellekirkwood marked this pull request as ready for review March 2, 2022 16:36
Copy link
Contributor

@kittykat kittykat left a comment

Choose a reason for hiding this comment

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

I think this is worth trying out!

@daniellekirkwood
Copy link
Contributor Author

@nadonomy I'm going to go ahead and merge this change as @kittykat has reviewed and we've talked about this on Element. Thanks

@daniellekirkwood daniellekirkwood merged commit 938eac4 into develop Apr 11, 2022
@daniellekirkwood daniellekirkwood deleted the daniellekirkwood-patch-190 branch April 11, 2022 16:54
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.

Write a Chat Effects strategy document for Element
2 participants