lemontree210 · GitHub

added 9 commits

November 5, 2022 14:01
* add constants for `max_connections`
* expand docstring for `secret_token`
* add limits for heading
* add minimum proximity_alert_radius
(as it is non-zero)
* fix docstrings that link to HEADING
constant twice instead of PROXIMITY_ALERT_RADIUS
(and hence 360 was displayed instead of 100000)
…terals-3107
as suggested in review for PR #3336,
but in singular (rather than `...Limits`)
to conform with review for #3343

Closed

* add links to/from `InlineQueryResultLocation` and `InputLocationMessageContent`
that were not added in previous commits
* note that `Location` class does not have
limits specified for `live_period` and
`proximity_alert_radius` but `InlineQueryResultLocation`
and `InputLocationMessageContent` do
…terals-3107

lemontree210

+ make sure the order of constants is kept in docs

Bibo-Joshi

note that this approach leads to similar constants
in multiple classes
Typo: "Minimum" instead of "Maximum"
reverting changes made to telegram.dice.rst
in   cbe20e6 

lemontree210

lemontree210

these are allowed lengths of strings, not allowed values

Merged

23 tasks

use existing class `BotCommandLimit`
minimum length is only specified in 1 class and 1 method

@lemontree210

* InlineQueryResultContact* InputContactMessageContent

@lemontree210

lemontree210

to eliminate ambiguity
…terals-3107

@lemontree210

harshil21

@harshil21 harshil21 linked an issue

Nov 21, 2022

that may be closed by this pull request

Closed

# Conflicts:
#	telegram/_bot.py
#	telegram/constants.py

@Bibo-Joshi

@Bibo-Joshi

Merged

Read the original on github.com ↗