Warning Points

[DubStepMad]

Allows warning points to be set in the players table with the same colum (warning) used in 'notes'. This does not create the colum in the 'players' table.

Other edit allows the warning points to be displayed on the users account information.

[DubStepMad]

I will try later when I get home to update it with the UID.

On Wed, May 3, 2017 at 10:29 PM, Jason_000 [email protected] wrote:

@Jason2605 commented on this pull request.

In notes.php https://github.com/Jason2605/AdminPanel/pull/28#discussion_r114658339:

 echo '<td>'."<input class='form-control' type=text name=note value=''> </td>";

 echo '<td>'."<input class='btn btn-primary btn-outline' type=submit name=update value=Update".'> </td>';

• echo "".'<input type=hidden name=hidden value='.$row['uid'].'> ';

• echo "".'<input type=hidden name=hidden value='.$row['playerid'].'> ';

I prefer the use of UID here over playerid due to 5.0 changing the table to be pid.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Jason2605/AdminPanel/pull/28#pullrequestreview-36143738, or mute the thread https://github.com/notifications/unsubscribe-auth/ANKRsLEV3hbpULLxhvRNdgYfXpiiiEQ6ks5r2PGngaJpZM4NFafn .

[DubStepMad]

I added row 83 to it for my own use, can be removed since it is not really needed.

Connor

On 10 May 2017 18:39, "Jason_000" [email protected] wrote:

@Jason2605 commented on this pull request.

In notes.php https://github.com/Jason2605/AdminPanel/pull/28#discussion_r115804879:

@@ -80,8 +80,8 @@ echo '

'; echo ''.$row['uid'].' '; echo ''.$row['name'].' ';

• echo ''.$row['aliases'].' ';
• echo ''."CommendationWarningCautionBig Caution ";

• echo ''.$row['playerid'].' ';

Same here

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Jason2605/AdminPanel/pull/28#pullrequestreview-37376388, or mute the thread https://github.com/notifications/unsubscribe-auth/ANKRsL6IMGBafIPPXBfrBPhvR5RGKQQTks5r4fY6gaJpZM4NFafn .

[itsJarrett]

Very interesting concept.

[itsJarrett]

@Jason2605 might want to make your decisions whether or not to include this. Other contributions to the repo are creating conflicts. 🕐