Skip to content

add User comment ban.#650

Open
RedLime wants to merge 2 commits intoCvolton:masterfrom
RedLime:comment-ban
Open

add User comment ban.#650
RedLime wants to merge 2 commits intoCvolton:masterfrom
RedLime:comment-ban

Conversation

@RedLime
Copy link
Contributor

@RedLime RedLime commented Aug 20, 2020

it needs toolCommentban role permission!
you can ban who people of upload wrong comment.
/database/tools/commentBan.php Through here, you can designate userID, duration and some reason to ban as you want.

and last, a comment ban's are contains 16 modaction id.

an image

comment ban
@EMREOYUN
Copy link
Contributor

Related with #644

@RedLime
Copy link
Contributor Author

RedLime commented Aug 21, 2020

@Cvolton can you commit it? 😀

@Wyliemaster
Copy link
Contributor

current version didnt work when i test, its because it lacks a few columns in the users table

i added this to the users table in the database.sql and it worked

commentBanDuration int(11) NOT NULL DEFAULT 0,
commentBanTime int(11) NOT NULL DEFAULT 0,
commentBanReason varchar(100) COLLATE utf8_unicode_ci NOT NULL DEFAULT 0,

@RedLime
Copy link
Contributor Author

RedLime commented Aug 24, 2020

current version didnt work when i test, its because it lacks a few columns in the users table

i added this to the users table in the database.sql and it worked

commentBanDuration int(11) NOT NULL DEFAULT 0,
commentBanTime int(11) NOT NULL DEFAULT 0,
commentBanReason varchar(100) COLLATE utf8_unicode_ci NOT NULL DEFAULT 0,

oh i'm so sorry. i forgot it :(
i will fix that

@EMREOYUN
Copy link
Contributor

current version didnt work when i test, its because it lacks a few columns in the users table
i added this to the users table in the database.sql and it worked
commentBanDuration int(11) NOT NULL DEFAULT 0,
commentBanTime int(11) NOT NULL DEFAULT 0,
commentBanReason varchar(100) COLLATE utf8_unicode_ci NOT NULL DEFAULT 0,

oh i'm so sorry. i forgot it :(
i will fix that

Fix that and include there, do not open a new Pull Request

@fancythedeveloper
Copy link

Merge now?

@RedLime
Copy link
Contributor Author

RedLime commented Aug 25, 2020

Merge now?

not yet

@RedLime
Copy link
Contributor Author

RedLime commented Sep 4, 2020

now i tested.
it ready to merge

@ssdrive-flaktra
Copy link

@Cvolton merge

@tzt345
Copy link

tzt345 commented Jun 9, 2021

@Cvolton merge it now!!!

@EMREOYUN
Copy link
Contributor

Don't force it!

@tzt345
Copy link

tzt345 commented Jun 10, 2021

Don't force it!

K

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants