python-telegram-bot / python-telegram-bot/ptbcontrib
[BUG] PostgresPersistence does not correcly load data from database.
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 118
- Forks
- 47
- PR merge metrics
- No merged PRs in 30d
Description
Module this issue is about
ptbcontrib/postgres_persistence/postgrespersistence.py
Steps to reproduce
-
Use specified module as persistence class.
-
Build your application loading the persistence.
-
Add some random data inside
bot_data, as it is the only one usable before getting updates. -
Generate an update so the persistence is updated in the database.
-
Restart the bot script.
Expected behaviour
Added data (step 3) should be correctly loaded from database, so *_data dictionaries should contain the same information they had before restarting bot script.
Actual behaviour
Even if database is actually updated, *_dict are empty, so persistence is not restored correctly.
Configuration
Operating System:
Distributor ID: Linuxmint
Release: 21.3
Version of Python, python-telegram-bot & dependencies:
Python 3.12
PTB 21.2
Please Note
This is the first time I'm trying to contribute, so sorry if something is not clear enough.
Contributor 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.
Research direction
Start in ptbcontrib/postgres_persistence/postgrespersistence.py and reproduce the reported sequence: add data to bot_data, generate an update, then restart the bot. Trace the database loading path and verify that the *_data dictionaries contain the same values after restart; done means persistence data is restored instead of remaining empty.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- postgresql, python
- Domain
- databases
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100