Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PlayerJoinEvent Message #26

Closed
MalakasPlayzMC opened this issue Feb 3, 2018 · 12 comments
Closed

PlayerJoinEvent Message #26

MalakasPlayzMC opened this issue Feb 3, 2018 · 12 comments
Assignees

Comments

@MalakasPlayzMC
Copy link

MalakasPlayzMC commented Feb 3, 2018

It is not an issue. It's a request. Can you add that on the join event?

$player = $event->getPlayer();
$name = $player->getName();
$text[0] = TextFormat::DARK_PURPLE . "##########################";
$text[1] = TextFormat::AQUA . "Welcome, $name";
$text[2] = TextFormat::LIGHT_PURPLE . "Website: " . TextFormat::WHITE . "Customizable via config";
$text[3] = TextFormat::LIGHT_PURPLE . "Vote Website " . TextFormat::WHITE . "Customizable via config";
$text[4] = TextFormat::AQUA . "To Login do /login . To Register do /register ";
$text[5] = TextFormat::DARK_PURPLE . "##########################";
$player->sendMessage($text[0]);
$player->sendMessage($text[1]);
$player->sendMessage($text[2]);
$player->sendMessage($text[3]);
$player->sendMessage($text[4]);
$player->sendMessage($text[5]);

@HyGlobalHD
Copy link
Member

does the /login and /register is needed? cause there already xBox Auth

@HyGlobalHD
Copy link
Member

I can add this but the plugin is in re-write progress, so it might be none usable for a while.

@MalakasPlayzMC
Copy link
Author

The plugin is not for API 3.0.0 (according to readme.md). X-Box Auth is required on MCPE 1.2 and higher if I am not false.

@HyGlobalHD
Copy link
Member

HyGlobalHD commented Feb 15, 2018

Well This Plugin Is OUTDATED, And In Re-Write Branch , I Currently Fixing Some Stuff But I Tried To Update It To The Latest API . Also I'll Tried To Update It For MCPE 1.2 And Higher, Which The Auth Is Not Needed;
However, If You Really Need The Auth You Can Request It.
Also Yes, According To README.md The Plugin Is Not For API 3.0.0 Or Higher Cause It's Still OUTDATED.

@MalakasPlayzMC
Copy link
Author

I do not need the auth. Forget the $text[4]. What about the other messages?

@NeuroBinds
Copy link
Contributor

Sure.

@NeuroBinds
Copy link
Contributor

Anywhere, is this request to add for the rewrite branch or in the current master?

@MalakasPlayzMC
Copy link
Author

Rewrite, the master is discontinued

@MalakasPlayzMC
Copy link
Author

Bump

@Cynoser
Copy link
Member

Cynoser commented Mar 21, 2018 via email

@AcNEO
Copy link
Contributor

AcNEO commented Mar 24, 2018

please take note that this plugin wouldn't be update to frequently as stated in README.md

View https://github.com/BackToAction/ARCore/blob/master/README.md#note

@BackToAction BackToAction deleted a comment from MalakasPlayzMC Jun 18, 2018
@BackToAction BackToAction deleted a comment from MalakasPlayzMC Jun 18, 2018
@BackToAction BackToAction deleted a comment from NeuroBinds Jun 18, 2018
@AcNEO
Copy link
Contributor

AcNEO commented Jun 18, 2018

close the issues, cause we're not sure to update it anymore.
I will reconsidered to add this if I had time, cause we're all very busy with RL....
sorry for the inconvenience.

@AcNEO AcNEO closed this as completed Jun 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants