Hay1tsme/artemis

ongeki: fix clearstatus type #147

Merged

Midorica merged 2 commits from akanyan/artemis:fix/ongeki/clearstatus into develop

2024-06-08 15:34:25 +00:00

akanyan commented

2024-05-26 21:45:22 +00:00

Contributor

~~Partially~~ fixes #146 *.

* Users have to run https://gitea.tendokyu.moe/akanyan/artemis-ongeki-fix-clear-status to heal the database.

~~Partially~~ fixes #146 \*. \* Users have to run https://gitea.tendokyu.moe/akanyan/artemis-ongeki-fix-clear-status to heal the database.

akanyan referenced this pull request

2024-05-26 21:52:16 +00:00

Ongeki: Folder and overall clear counts don't save/display properly #146

Midorica commented

2024-05-28 01:59:22 +00:00

Collaborator

I can indeed confirm the value that was set before was incorrect

did you want to push another fix or should we merge it now?

I can indeed confirm the value that was set before was incorrect did you want to push another fix or should we merge it now?

akanyan commented

2024-05-28 18:05:18 +00:00

Author

Contributor

should we merge it now?

Yes, I realized there is nothing else to fix on the side of ARTEMiS.

> should we merge it now? Yes, I realized there is nothing else to fix on the side of ARTEMiS.

Hay1tsme commented

2024-05-29 02:35:22 +00:00

Owner

Can you pull down latest develop and run python dbutils.py upgrade and python dbutils.py create-autorevision to create a fix on the database side?

Can you pull down latest develop and run `python dbutils.py upgrade` and `python dbutils.py create-autorevision` to create a fix on the database side?

Hay1tsme added the

labels 2024-05-29 02:35:39 +00:00

akanyan commented

2024-05-29 10:10:53 +00:00

Author

Contributor

It was already a tinyint on the database side, which was fine, no?

Done anyway.

It was already a tinyint on the database side, which was fine, no? Done anyway.

akanyan force-pushed fix/ongeki/clearstatus from 03cc636c25 to df2a4d3074

2024-05-29 10:16:02 +00:00

Compare

Midorica commented

2024-06-08 15:34:19 +00:00

Collaborator

Since you have included what Hay requested the merge will now be approved

Thank you!

Since you have included what Hay requested the merge will now be approved Thank you!

Midorica merged commit e6965b568d into develop

2024-06-08 15:34:25 +00:00

Midorica referenced this issue from a commit

2024-06-08 15:34:26 +00:00

Merge pull request 'ongeki: fix clearstatus type' (#147) from akanyan/artemis:fix/ongeki/clearstatus into develop

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No project

No Assignees

3 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Hay1tsme/artemis#147

No description provided.