Add Redis db support #18
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fixed a 'c.execute()' invoked with no query.
Closed
3 tasks
Added redis db support and rewrited all 'db.py' fuctions: 1. `__init__`: Initializes an instance of RedisRequestManager with a given Redis connection and index. 2. `add_request`: Adds a new request for a user, checks for duplicates, and sets the initial keys in Redis. 3. `delete_request`: Deletes all keys related to a specific user's request. 4. `update_request_status`: Updates the status of a request based on its current status. 5. `get_request_data`: Retrieves the data associated with a specific user's request, including start date, end date, and status. 6. `get_all_request_data`: Retrieves all request data for all users in Redis.
Added Redis ip and password to the configuration files for proper setup and usage of Redis in the application.
Added 'tests.py' file for testing Redis database functionality.
Now DBManager is created after loading configs from file because it needs the REDIS user and password.
Fixed wrong return type when no request found: now returns a '{}' (empty
dict).
Fixed a problem with updating request status: 'enddate' now is unset
when status is changed from 'approved' to 'pending' (revoke request).
Adapted routes data (dict keys) with new redis db.
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.