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

Allow Other Users In Groups #9

Merged
merged 4 commits into from
Feb 11, 2024
Merged

Allow Other Users In Groups #9

merged 4 commits into from
Feb 11, 2024

Conversation

beardedtim
Copy link
Member

Description

This allows us to add new people to the group messages. It starts with some clean up and refactoring of the UI to make it easier to iterate.

Tim Roberts added 2 commits February 10, 2024 17:20
This cleans up a lot of the hacks that we did just to get things
going along with updating the styling to make the Messages page
at least look presentable
This commit sends messages to groups and uses the Redux Toolkit
in order to store the state of the messages
@beardedtim beardedtim marked this pull request as ready for review February 11, 2024 01:43
Copy link

Coverage report for client

St.
Category Percentage Covered / Total
🟢 Statements 100% 8/8
🟢 Branches 100% 4/4
🟢 Functions 100% 2/2
🟢 Lines 100% 7/7

Test suite run success

4 tests passing in 1 suite.

Report generated by 🧪jest coverage report action from edb5d73

@beardedtim beardedtim merged commit 4f8275a into main Feb 11, 2024
2 checks passed
@beardedtim beardedtim deleted the feat/other-users branch February 11, 2024 01:50
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.

1 participant