Skip to content

add User comment ban. - #650

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

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

Conversation

@RedLime

@RedLime RedLime commented Aug 20, 2020

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Contributor

Related with #644

@RedLime

RedLime commented Aug 21, 2020

Copy link
Copy Markdown
Contributor Author

@Cvolton can you commit it? 😀

@Wyliemaster

Copy link
Copy Markdown
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

RedLime commented Aug 24, 2020

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown
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
Copy Markdown

Merge now?

@RedLime

RedLime commented Aug 25, 2020

Copy link
Copy Markdown
Contributor Author

Merge now?

not yet

@RedLime

RedLime commented Sep 4, 2020

Copy link
Copy Markdown
Contributor Author

now i tested.
it ready to merge

@ssdrive-flaktra

Copy link
Copy Markdown

@Cvolton merge

@tzt345

tzt345 commented Jun 9, 2021

Copy link
Copy Markdown

@Cvolton merge it now!!!

@EMREOYUN

Copy link
Copy Markdown
Contributor

Don't force it!

@tzt345

tzt345 commented Jun 10, 2021

Copy link
Copy Markdown

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