lnp2pBot / lnp2pBot/bot

Add an optional [community] param to /buy and /sell commands

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

Nobody has claimed this yet.

Dominant language
TypeScript
Stars
292
Forks
136
Avg merge
19m
Merged PRs (30d)
1

Description

I publish my orders in two communities, the default and a currency specific, so when it comes to create an order (buy or sell) I create one, switch community and the copy and paste it. This process is annoying and error prone.

I would be ideal if the order commands allows an optional [community], which when set instruct the bot to also publish the order to such community (along with the one that is currently set by the user).

Something like:

/sell <monto en sats> <monto en fiat> <código fiat> <método de pago> [prima/descuento] [community]

Contributor guide

Open the contributing guide

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 Telegram command handlers for /buy and /sell and read how the currently selected community is used when publishing orders. Add optional community handling for both commands, preserving publication to the selected community and also publishing to the supplied one; verify both buy and sell flows.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
backend
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
64/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.