stpain / guildbook

Guildbook
https://www.curseforge.com/wow/addons/guildbook
zlib License
9 stars 9 forks source link

Main Spec on Guildwindow not reflecting current spec #53

Open emanueol opened 3 years ago

emanueol commented 3 years ago

Hi,

Im guildmaster and asked guildies to install Guildbook, and we could see on the new bigger Guild Window the column "Main Spec" and even the professions.

Today a guildie told he respecdd from Restoration (shaman) into Dps (either Elemental or Enchancement), but Guild window column still showing "Restoration", instead of showing "Elemental" or "Enhancement".

Is the Main Spec supposed to be automatic refresh, do does it need some manual clicks ?

FYI: I tried opened Roster, clicked guildie chars and clicked Reset, but still mains spec remain old one. As side-effect 2 other guildies now got their mains spec and professions cleared out.

I really would like to use this addon for guild management, but i keep bouncing into basic foundational problems that will make this addon unusable as will be hard to keep explaining workarounds to each new problem each new guildie encounters. If addon not rock solid then guildies will not use it.. and im already started finding resistence.

So, look forward to understand the "Main Spec changed/refresh" issue solution.

Thanks. ~Budha~

stpain commented 3 years ago

The addon will assign whichever spec has the most points if a spec has not been set, after that it requires a manual change in the profile section. i will however take a look at what events i can listen to for these situations.

emanueol commented 3 years ago

Ok I understand.

But my guildie told he just manually set on his profile "Elemental", he can also see the change in guild window, but my self on both guild window and checking roster i still see old Main Spec of my guildie.

Thanks//Kind Regards, Emanuel Oliveira

On Wed, 25 Aug 2021 at 19:38, stpain @.***> wrote:

The addon will assign whichever spec has the most points if a spec has not been set, after that it requires a manual change in the profile section. i will however take a look at what events i can listen to for these situations.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/stpain/guildbook/issues/53#issuecomment-905779821, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACAIIDCVBRKR5BFQRTGG3Q3T6U2DLANCNFSM5CZWCQ2Q .

stpain commented 3 years ago

Well i've found an event, my issue now is making sure its not going to spam the chat comms channels. If you click to view the persons profile do you get the updated spec?

emanueol commented 3 years ago

I dont see guildie main spec on my side (guild window and/or guildbook > roster). Only guildie that changed spec sees his new spec on guildwindow and roster and only after manually setting on his Profile th email spec.

Hope this helps.

Thanks//Kind Regards, Emanuel Oliveira

On Wed, 25 Aug 2021 at 22:54, stpain @.***> wrote:

Well i've found an event, my issue now is making sure its not going to spam the chat comms channels. If you click to view the persons profile do you get the updated spec?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/stpain/guildbook/issues/53#issuecomment-905902395, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACAIIDHKUJCAVV4CTKBE43LT6VRBZANCNFSM5CZWCQ2Q .

stpain commented 3 years ago

hmm, ok well either ive had a quick test with a new system which should resolve this issue.

emanueol commented 3 years ago

RESPET BROADAST: Can addon detect talents change finished (number of free talents points to assign = zero) and then broadcast main spec to other guildies using guildbook ? Whats best place for me to request this enhancement, as im having hardtime having guildies to adhere to using Guildbook, since it needs too much manual a actions for them to syn up thing, namely: recipes and mainspec.

stpain commented 3 years ago

The addon will check what points have been spent in which tree and where a guild member HASN'T set a spec, the addon will take a guess based on which tree has most points.

However, as with classic, TBC doesnt have a strict class>spec system (unlike retail). There are specs that dont fit points per tree and so character spec has been left as a player option. Personally opening your profile and selecting from a dropdown isnt a taxing requirement (as spec doesnt change often)

I could add a check box and option to ONLY use the spec which has the most points.

When/if we get a classic wrath then its much easier to say the tree with most points is going to be the characters spec.

Warcraft logs (iirc) has recently added new specs such as shaman tank (warden) and a couple warrior options i think.

Also, as a player levels up they may show with an incorrect spec if say a priest takes wand spec but then goes into shadow.

Its a tricky area tbh.

stpain commented 3 years ago

Recipes will sync over time as players open their profession windows, there is the option to share a 'snapshot' of your db via the import/export interface.

emanueol commented 3 years ago

Let me be more clear:

ITs getting really frustrating this.

Thanks//Kind Regards, Emanuel Oliveira

On Wed, 8 Sept 2021 at 16:36, stpain @.***> wrote:

Recipes will sync over time as players open their profession windows, there is the option to share a 'snapshot' of your db via the import/export interface.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/stpain/guildbook/issues/53#issuecomment-915347081, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACAIIDBGOH42FQG35EWYDMTUA57GZANCNFSM5CZWCQ2Q .

stpain commented 3 years ago

does everyone have the latest version?

i am going to have to try and replicate this issue as its not something ive seen before.

have you tried to set/change specs with the debug window open to see if any comms happen?

i am working on adding an option to let the addon set your main spec, but if you're saying no comms come through then there is a different cause for your issues.

emanueol commented 3 years ago

I never used debug option, but going try. As myself software engineer shall not be hard enabling debug and try confirm why no comms.

Thanks and Kind regards, Emanuel Oliveira Senior Data Engineer

On Thu, Sep 9, 2021, 01:32 stpain @.***> wrote:

does everyone have the latest version?

i am going to have to try and replicate this issue as its not something ive seen before.

have you tried to set/change specs with the debug window open to see if any comms happen?

i am working on adding an option to let the addon set your main spec, but if you're saying no comms come through then there is a different cause for your issues.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/stpain/guildbook/issues/53#issuecomment-915665312, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACAIIDAZVRBRNEYMXLXVFOLUA76CZANCNFSM5CZWCQ2Q .

stpain commented 3 years ago

mainspec-comms

so you should see something like this, this is from the latest version on a character i ccreated earlier this evening.

emanueol commented 3 years ago

Awesome, thanks, kook forward to test/debug.

btw: How can i create an animated gif like that of yours? And deems ots hosted on url Google content ? i wonder whats this service/how to upload gif? i got google drive but unsure if related 😊

Thanks and Kind regards, Emanuel Oliveira Senior Data Engineer

On Thu, Sep 9, 2021, 01:59 stpain @.***> wrote:

[image: mainspec-comms] https://user-images.githubusercontent.com/47498931/132604903-220e86b8-8cdf-4c22-b486-99a3b68ee985.gif

so you should see something like this, this is from the latest version on a character i ccreated earlier this evening.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/stpain/guildbook/issues/53#issuecomment-915676809, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACAIIDGR3QT7PTMT5XWY5QDUBABHPANCNFSM5CZWCQ2Q .

stpain commented 3 years ago

its an app/program called ScreenToGif, then just drag and drop the image file here, probably has a size limit though

emanueol commented 3 years ago

Thanks, I'll check it out, and create a similar gif showing 2 debugs:

  1. me changing main spec and watching comms
  2. will ask guildie to do same and see on my side if i see something coming.

Makes sense these 2 tests i guess.

Thanks and Kind regards, Emanuel Oliveira Senior Data Engineer

On Thu, Sep 9, 2021, 02:06 stpain @.***> wrote:

its an app/program called ScreenToGif, then just drag and drop the image file here, probably has a size limit though

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/stpain/guildbook/issues/53#issuecomment-915679671, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACAIIDHKC3ZJJFKTQ6HXFDLUBACBFANCNFSM5CZWCQ2Q .