-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
feat(messaging): ui #25811
feat(messaging): ui #25811
Conversation
Size Change: +316 B (+0.03%) Total Size: 1.15 MB ℹ️ View Unchanged
|
📸 UI snapshots have been updated3 snapshot changes in total. 0 added, 3 modified, 0 deleted:
Triggered by this commit. |
📸 UI snapshots have been updated2 snapshot changes in total. 0 added, 2 modified, 0 deleted:
Triggered by this commit. |
4da8f83
to
df08675
Compare
📸 UI snapshots have been updated1 snapshot changes in total. 0 added, 1 modified, 0 deleted:
Triggered by this commit. |
📸 UI snapshots have been updated1 snapshot changes in total. 0 added, 1 modified, 0 deleted:
Triggered by this commit. |
📸 UI snapshots have been updated2 snapshot changes in total. 0 added, 2 modified, 0 deleted:
Triggered by this commit. |
📸 UI snapshots have been updated2 snapshot changes in total. 0 added, 2 modified, 0 deleted:
Triggered by this commit. |
Problem
Splitting out work from #25719
Changes
Adds a feature flag
messaging
which exposes the following UI. Everything works except for actually testing.(the spinner froze in the end is because locally I had just merged from master before recording the clip, and seems like the endpoint required a pip package that wasn't installed)
How did you test this code?
Clicked around in the UI, everything worked as expected other than what's coming in the next PR (actual testing).
All old hog functions also seem to work.