Open SGA-A opened 4 months ago
Implementing loggers will only get harder as development continues, the sooner this is done the better. There should ideally be a single method that would always be used to log this info so that for instance, function renames aren't needed on a large scale.
On second thought, I am considering implementing a maximum row limit per user ID. Currency logs will be exceptionally big. I'll need to figure out how to do this first. But, this is not a huge priority, as outlined in the project.
Starting soon, trade requests will appear in notifications.
A single notification can contain the following:
Few commands to manage your notifications will be held in the notfications group.
/notifications view (id)
/notifications list
The target is to have all of the following be stored in notifications:
Trade requests eventually expire so a background task will need to be created to delete expired requests after the expiry timestamp.
New commands will be created to accept and reject trade requests.
Soon, trade requests will appear in notifications.
A single notification can contain the following:
Few commands to manage your notifications will be held in the notfications group.
/notifications view (id)
/notifications list
The target is to have all of the following be stored in notifications:
Trade requests eventually expire so a background task will need to be created to delete expired requests after the expiry timestamp.
New commands will be created to accept and reject trade requests.
The gist of what will be logged
At the moment, we have no plans to limit the amount of logs shown, but it is something we will keep in mind in the future if it becomes a problem. For reference, Dank Memer's notification lists are limited to a maximum of 200 stored per user. This number was much higher just a few years ago when the bot had less traction.
For logging mechanisms and notifications, there will be a paginator and it will have an index tracker (using the
Pagination
class).For notifications, you will be able to access a specific notification via it's an ID which opens up the notification and a paginated list of the notification directly. This means when using the command to view a specific notification (
/notification view id:55
), it is equivalent to (/notification list
) but with the paginator length set to 1, and the page index being 55 (or 56?). We would also need to implement a search subcommand, to search for a notification by keyword (checking if it is present in both the notification title)For the currency log, there should also be a way of sorting by the methods of how the currency entered/left in the form of a dropdown.
What it could contain
A notification typically has the following attributes:
A currency log typically has the following attributes: