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

Added xero.gg support. #2451

Merged
merged 12 commits into from
Nov 11, 2020
Merged

Added xero.gg support. #2451

merged 12 commits into from
Nov 11, 2020

Conversation

Dekirai
Copy link
Contributor

@Dekirai Dekirai commented Nov 9, 2020

Showcase: Showcase
Website: xero.gg
What it is: xero.gg is a player and clan database for the private server called "S4 Xero" of the base game "S4 League".
What the presence does: The presence mainly shows what profile or clan you are currently looking at or what news you are currently reading. More features included and might follow.

Screenshots of the presence:
image
image
image

Copy link
Contributor

@rxri rxri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

upload proof of working presence

rxri
rxri previously approved these changes Nov 10, 2020
@Dekirai Dekirai closed this Nov 11, 2020
@Dekirai Dekirai reopened this Nov 11, 2020
Copy link
Contributor

@rxri rxri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You don't need let variables for them, because you only use these variables in if statements so please change it to const.

@Dekirai Dekirai requested a review from rxri November 11, 2020 14:42
@Dekirai
Copy link
Contributor Author

Dekirai commented Nov 11, 2020

I hope it's okay now..

Comment on lines 53 to 63
const claninfo = [
document.querySelector(
"#player-profile-header-heading > div.medium.normal-color-name"
).textContent,
document.querySelector(
"#clan-data-container > div > div.col-sm-3 > div > div.xero-pane-body > ul > li"
).lastChild.textContent,
document.querySelector(
"#clan-data-container > div > div.col-sm-9 > div:nth-child(2) > div > div > a > div.media-body.ml-2 > div.bold"
).textContent
];
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thats an array, thats now how you merge consts

const var1 = obj,
var2 = obj2;

@Dekirai
Copy link
Contributor Author

Dekirai commented Nov 11, 2020

Okay I hope I finally understood it now.
I feel so stupid, I'm so sorry.

@Dekirai Dekirai requested a review from Bas950 November 11, 2020 14:59
@Bas950
Copy link
Member

Bas950 commented Nov 11, 2020

just to be sure please upload new proof of it working

@Dekirai
Copy link
Contributor Author

Dekirai commented Nov 11, 2020

Ah sure, 1 sec
Video or images?

@Dekirai
Copy link
Contributor Author

Dekirai commented Nov 11, 2020

@Bas950 I replaced the Showcase Link with a new video (You can also see the current date and time of windows)

@mergify mergify bot merged commit 94136f7 into PreMiD:master Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants