update jsonrpc to new `get_summary_text()`
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 45/100
Research direction
Start with the reasoning in pull request #8813, linked from the issue body, to learn the new get_summary_text() signature and its three parameters. Then find the jsonrpc code that calls the summary API, check whether get_summary_text() already exists or whether get_summary() is what jsonrpc uses now, and update the call sites. Done means the jsonrpc layer compiles against the new API and its tests pass. Confirm the approach with a maintainer first, since the issue leaves open whether to add or reuse the function.
Written by the indexing model from the issue text.
Description
jsonrpc needs to be updated to new get_summary_text() api, see https://github.com/chatmail/core/pull/8813 for details and reasoning.
-
if there is already a
get_summary_text(), maybe we can add the three parameters -
if there is no
get_summary_text()yet, add it (might beget_summary()was used, that returns a lot of other things but does not accept parameters)
- Dominant language
- Rust
- Stars
- 937
- Forks
- 150
- Avg merge
- 1d 12h
- Merged PRs (30d)
- 90
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from chatmail/core
-
Difficulty 1/5 Under an hour Newbie friendliness 68/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 50/100
Maintainers usually reply within 1 day
-
Difficulty 4/5 3-5 days Newbie friendliness 25/100
Maintainers usually reply within 1 day
-
Difficulty 4/5 3-5 days Newbie friendliness 54/100
chatmail/core#8814 · 3 comments · 1 reaction ·
Maintainers usually reply within 1 day
-
bug
Difficulty 3/5 1-2 days Newbie friendliness 56/100
chatmail/core#8802 · 2 comments · 1 reaction ·
Maintainers usually reply within 1 day
Similar issues
-
Difficulty 1/5 Under an hour Newbie friendliness 88/100
pop-os/cosmic-comp#2956 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
libp2p/rust-libp2p#6694 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
EmbarkStudios/cargo-deny#910 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
tursodatabase/turso#10054 ·
Maintainers usually reply within 1 day
-
build: Update rust versionPossibly taken A pull request linked to this issue is open or already merged. Opendependencies
Difficulty 2/5 1-3 hours Newbie friendliness 67/100
ibm-granite-community/granite-cli#229 ·
Maintainers usually reply within 1 day