puregarlic/microclimate
3
1
Fork 0
Code Issues 33 Pull requests 1 Projects 2 Releases Packages 1 Wiki Activity Actions

26-channel-display-tauri #24

Merged
seb merged 7 commits from 7-channel-display-sidebar-two into issues/10-server-url 2026-02-17 15:33:51 -08:00
Conversation 2 Commits 7 Files changed 6 +183 -10
seb commented 2026-02-12 22:21:58 -08:00
Collaborator
Copy link

Closes #26.

This PR implements a client for connecting to the channel server, registers it with the AppState, and exposes the behavior through Tauri commands.

Closes #26. This PR implements a client for connecting to the channel server, registers it with the AppState, and exposes the behavior through Tauri commands.
Working Set 1
seb changed title from WIP: 7-channel-display-sidebar-two to WIP: 26-channel-display-tauri 2026-02-13 23:25:45 -08:00
seb commented 2026-02-13 23:26:09 -08:00
Author
Collaborator
Copy link

Don't merge this until #23 is in main. Or I could rebase and merge it into #23. Up to @tepichord

Don't merge this until #23 is in main. Or I could rebase and merge it into #23. Up to @tepichord
seb changed target branch from main to issues/10-server-url 2026-02-13 23:29:19 -08:00
seb changed target branch from issues/10-server-url to main 2026-02-13 23:31:18 -08:00
seb changed title from WIP: 26-channel-display-tauri to 26-channel-display-tauri 2026-02-13 23:32:23 -08:00
seb changed target branch from main to issues/10-server-url 2026-02-13 23:33:16 -08:00
puregarlic approved these changes 2026-02-14 13:30:12 -08:00
puregarlic left a comment
Owner
Copy link

LGTM. Just posting for posterity that the naming convention for MicroclimateClient will probably change in the future, since there are likely to be multiple clients (for multiple gRPC services). At that time, we'll want to move the pub mod proto definition.

LGTM. Just posting for posterity that the naming convention for `MicroclimateClient` will probably change in the future, since there are likely to be multiple clients (for multiple gRPC services). At that time, we'll want to move the `pub mod proto` definition.
server/build.rs Outdated
@ -2,2 +1,2 @@
tonic_prost_build::compile_protos("../proto/channels.proto")?;
Ok(())
fn main() {
tonic_prost_build::configure()
puregarlic commented 2026-02-14 08:57:15 -08:00
Owner
Copy link

Message types aren’t going to be JSON serialized and deserialized on the server side, so is it necessary to change the build code here?

Message types aren’t going to be JSON serialized and deserialized on the server side, so is it necessary to change the build code here?
puregarlic marked this conversation as resolved
seb force-pushed 7-channel-display-sidebar-two from bc1310b12a to f7ff0c7f89 2026-02-17 15:33:07 -08:00 Compare
seb merged commit 5e662a432e into issues/10-server-url 2026-02-17 15:33:51 -08:00
seb referenced this pull request from a commit 2026-02-17 15:33:52 -08:00
26-channel-display-tauri (#24)
puregarlic referenced this pull request from a commit 2026-02-18 17:17:19 -08:00
26-channel-display-tauri (#24)
Sign in to join this conversation.
Reviewers
No reviewers
tepichord
puregarlic
Labels
Clear labels   
area
client
Concerns the Tauri app

  
area
server
Concerns the server component

  
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
ui

Involves updating the client UI

  
wontfix

This won't be fixed

No labels
area
client
area
server
bug
duplicate
enhancement
help wanted
invalid
question
ui
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
puregarlic/microclimate!24
No description provided.