Some feedback

  • Resolved swerenfl

    (@swerenfl)


    8 years, 6 months ago

    Hi avenirer,

    Great addon. I use it here: https://lasthopeguild.com/roster/. I do have some feedback though, and I hope that with my feedback you can make the addon even better.

    First off let’s discuss the nuances. Everytime I download an update, I have to re-modify wowpi_shortcodes.php because there is some code that is not en friendly, and also code that doesn’t present well:
    1.) On line 195, I have to manually change eu to us. This should be variablized somewhere. PS – I have en-US selected in the options
    2.) Also on line 195, I have to change simple to advanced. Personal preference as going to a ‘simple’ armory is rather boring. Again maybe an option?
    3.) On line 200 I have to change eu to us on the img src link for ‘icon-guildmaster.gif’. The eu ref doesn’t work.
    4.) On line 200 I have to change ‘Guild master’ to ‘Guild Master’. Two upper letters is how it is in game.

    Secondly, I’d like to offer a suggestion:
    1.) Can you paginate it? Scrolling down a long list is tiring. The addon I used previously was paginated. The name of that addon for reference is: https://www.ads-software.com/plugins/world-of-warcraft-armory-table/.
    2.) A total count at the bottom or if pagination is added then on each page:
    i) Non pagination: 202 members
    ii) If pagination: Displaying 1 to 20 of 202 members
    3.) CSS/coloring the rows. It’s too plain. Would like to add color.

    The reason I switched addons is because with the aforementioned I had to force an update manually to refresh the roster. Does yours update periodically (ie: every 24 hours — I thought I saw this in the config; but was wondering if it really worked)?

    Thanks,
    Rich

    https://www.ads-software.com/plugins/wowpi/

Viewing 1 replies (of 1 total)
  • Plugin Author avenirer

    (@avenirer)

    Thank you very much for your input. Will do the necessary changes. It’s a bit hard for me to check for us servers as my characters are on eu servers. Maybe you could send me the battle.net credentials (secret keys, realm, character name, guild) for a us server in order for me to check if it all works?

    Also, I noticed some errors in the code when it checks for caching, hence it didn’t do it. In the next subversion will update that. Thank you, again.

Viewing 1 replies (of 1 total)
  • The topic ‘Some feedback’ is closed to new replies.