Penalty Improvement - Traceback

0x4007 commented 1 year ago

Dependencies

settlementIds as we currently do not have a table for those. https://github.com/ubiquity/ubiquibot/issues/787

Overview

Let's name this table Penalties to represent instances where a contributor is penalized. This table keeps a record of the exact comment or incident due to which a penalty was applied.

Penalties

A ledger of penalties applied to contributors. This table provides a link to the exact comment or reason for the penalty, making the process transparent and traceable.

Schema

Field	Type	Description	Example Value
`id`	Integer	Unique identifier for the penalty log entry.	1
`created`	DateTime	Timestamp when the penalty was logged.	2023-09-14T19:20:23.004Z
`updated`	DateTime	Timestamp when the penalty log was last updated.	2023-09-14T19:20:23.004Z
`userId`	Integer	GitHub `id` of the user penalized.	139262667
`settlementId`	Integer	Reference to the `Settlements` table.	512
`orgId`	Integer	GitHub `id` of the organization.	76412717
`repoId`	Integer	GitHub `id` of the repository where the issue is located.	534616569
`issueId`	Integer	GitHub `id` of the issue related to the penalty.	1868425596
`commentURL`	String	URL of the comment or incident leading to the penalty.	https://github.com/ubiquity/ubiquibot/issues/680#issuecomment-1718491622

Notes:

The commentURL field allows quick access to the exact comment or reason for the penalty.
I also made up the settlementIds as we currently do not have a table for those. However its spec is here https://github.com/ubiquity/ubiquibot/issues/787
Ensure foreign key constraints are set up properly to maintain data integrity.
We should rely on GitHub API's id properties instead of node_id Source.

Source

https://github.com/ubiquity/ubiquibot/issues/680#issuecomment-1718488900

wannacfuture commented 12 months ago

/start

ubiquibot[bot] commented 12 months ago

Too many assigned issues, you have reached your max of 2

seprintour commented 11 months ago

/start

ubiquibot[bot] commented 11 months ago



  
    
    
  
  
    Deadline
    Tue, 19 Sep 2023 22:43:15 UTC
  
  
    Registered Wallet
    0x3623338046b101ecEc741De9C3594CC2176f39E5

Tips:

Use /wallet 0x0000...0000 if you want to update your registered payment wallet address @user.
Be sure to open a draft pull request as soon as possible to communicate updates on your progress.
Be sure to provide timely updates to us when requested, or you will be automatically unassigned from the bounty.

seprintour commented 11 months ago

We'll probably need to change the schema to support a new table that has all the debt information. Basically a ledger of credits and debits which also associates the issue with each credit or debit. Example:

@pavlovcik Maybe we can have another table called offset that is an array of repo that offsets a particular penalty

created	updated	user	credit	org	repo	task	offset
2023-09-14T19:20:23.004Z	2023-09-14T19:20:23.004Z	whilefoo	-100	ubiquity	ubiquibot	680	566, 447
2023-09-14T19:20:32.554Z	2023-09-14T19:20:32.554Z	whilefoo	150	ubiquity	ubiquity-dollar	733	234, 444, 560

0x4007 commented 11 months ago

@seprintour

Storing multidimensional data in a SQL database cell should be avoided
Also I updated the spec.

wannacfuture commented 11 months ago

I think penalty_amount field should be added. @pavlovcik

seprintour commented 11 months ago

/stop

ubiquibot[bot] commented 11 months ago

You have been unassigned from the bounty @seprintour

0x4007 commented 11 months ago

Rolled into https://github.com/ubiquity/ubiquibot/issues/787

ubiquibot[bot] commented 11 months ago

! Error: Permit generation disabled because this is marked as unplanned.

ubiquity / ubiquibot