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

feat: add raid party members to loot metadata #478

Merged
merged 9 commits into from
Aug 5, 2024

Conversation

iProdigy
Copy link
Member

Closes #477

@iProdigy iProdigy marked this pull request as ready for review May 19, 2024 09:49
Copy link
Member

@pajlada pajlada left a comment

Choose a reason for hiding this comment

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

Code looks great 👍 Haven't tested in-game

Copy link
Member

@Felanbird Felanbird left a comment

Choose a reason for hiding this comment

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

i don't have a raiding gang laying around either so we'll just assume it works

@iProdigy
Copy link
Member Author

@zneix would you be willing to test this branch next time u do cox & toa? (essentially just trigger the loot notification & lepp can grab the json to verify)

shadow jar: https://github.com/pajlads/DinkPlugin/actions/runs/9146741541/artifacts/1516909819

@zneix
Copy link

zneix commented May 21, 2024

Sure thing, I'll give it a show in the coming days.

@zneix
Copy link

zneix commented May 27, 2024

From a casual solo ToA run I've managed to produce this payload https://paste.ivr.fi/raw/cywafugavo (copied below)

{"type":"LOOT","playerName":"zneix","accountType":"NORMAL","dinkAccountHash":"e3791a53061d0458cd4e3a080277e36d7b871d12c4642be8377b77c3","clanName":"Tench","seasonalWorld":false,"extra":{"items":[{"id":5935,"quantity":29,"priceEach":241,"name":"Coconut milk"},{"id":1391,"quantity":29,"priceEach":7971,"name":"Battlestaff"},{"id":560,"quantity":1660,"priceEach":113,"name":"Death rune"}],"source":"Tombs of Amascut","category":"EVENT","killCount":164,"party":["zneix"]},"discordUser":{"id":"288028423031357441","name":"zneix","avatarHash":"89a62e8d973219b4abd89769057a0f03"},"content":"zneix has looted: \n\n29 x Battlestaff (231K)\n1660 x Death rune (187K)\nFrom: Tombs of Amascut","embeds":[]}

P.S. Interesting observation is that the field killCount says 164 which is my Normal ToA KC, however this was an expert mode raid I've completed, of which, my KC was 219 (220 with this raid). That is of course irrelevant, but probably something worth to keep in mind.
game message with toa KC

@zneix
Copy link

zneix commented May 28, 2024

Another test, in a duo ToA this time https://paste.ivr.fi/raw/epasurydus (copied below)

{"type":"LOOT","playerName":"zneix","accountType":"NORMAL","dinkAccountHash":"e3791a53061d0458cd4e3a080277e36d7b871d12c4642be8377b77c3","clanName":"Tench","seasonalWorld":false,"extra":{"items":[{"id":6016,"quantity":38,"priceEach":1943,"name":"Cactus spine"},{"id":26390,"quantity":2,"priceEach":59362,"name":"Blood essence"},{"id":27293,"quantity":1,"priceEach":100,"name":"Cache of runes"}],"source":"Tombs of Amascut","category":"EVENT","killCount":164,"party":["zneix","PolskaPride"]},"discordUser":{"id":"288028423031357441","name":"zneix","avatarHash":"89a62e8d973219b4abd89769057a0f03"},"content":"zneix has looted: \n\n38 x Cactus spine (73.8K)\n2 x Blood essence (118K)\n1 x Cache of runes (100)\nFrom: Tombs of Amascut","embeds":[]}

@zneix
Copy link

zneix commented Jun 1, 2024

Interestingly enough the CoX seems to not function properly with the raid members https://paste.ivr.fi/raw/abysovasiz (copied below)
this was a regular mode raid with 6 people

{"type":"LOOT","playerName":"zneix","accountType":"NORMAL","dinkAccountHash":"e3791a53061d0458cd4e3a080277e36d7b871d12c4642be8377b77c3","clanName":"Tench","seasonalWorld":false,"extra":{"items":[{"id":447,"quantity":774,"priceEach":103,"name":"Mithril ore"},{"id":2485,"quantity":99,"priceEach":1438,"name":"Grimy lantadyme"}],"source":"Chambers of Xeric","category":"EVENT","killCount":44,"party":[]},"discordUser":{"id":"288028423031357441","name":"zneix","avatarHash":"89a62e8d973219b4abd89769057a0f03"},"content":"zneix has looted: \n\n99 x Grimy lantadyme (142K)\nFrom: Chambers of Xeric","embeds":[]}

@iProdigy iProdigy marked this pull request as draft June 3, 2024 22:32
@iProdigy
Copy link
Member Author

iProdigy commented Aug 5, 2024

I just tested CoX - jagex doesn't populate the widget unless the Grouping tab is selected (but can be minimized)

with the tab selected, it looks like the code obtains the names correctly

image

@iProdigy iProdigy marked this pull request as ready for review August 5, 2024 02:04
@iProdigy iProdigy merged commit 79c9ce5 into pajlads:master Aug 5, 2024
4 checks passed
@iProdigy iProdigy deleted the feature/loot-raid-party branch August 5, 2024 02:05
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.

Send group data when getting drop from group content
4 participants