liveview-native / liveview-native/liveview-native-core

Remove LiveSocket api.

Open
#311 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

area/ffi good first issue
Dominant language
Rust
Stars
166
Forks
14
PR merge metrics
No merged PRs in 30d

Description

The current `LiveSocket` object is being retired in favor of the `LiveViewClient` - we need to remove all of the old unused code except for the `LiveChannel` - which should likely be refactored to respect it's new purpose as an internal piece of API infrequently exposed to the user.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the Rust definitions and references for LiveSocket, LiveViewClient, and LiveChannel, then determine which old code is unused. Remove the retired LiveSocket API, retain LiveChannel, and refactor it toward its stated internal purpose; done means no remaining LiveSocket dependencies and the project still builds and tests successfully.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
api
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.