+
Skip to content
This repository was archived by the owner on Sep 17, 2024. It is now read-only.

weak auras fixes #470

Merged
merged 3 commits into from
Jan 4, 2021
Merged

weak auras fixes #470

merged 3 commits into from
Jan 4, 2021

Conversation

tarkah
Copy link
Member

@tarkah tarkah commented Jan 4, 2021

Resolves #465, #466

Proposed Changes

  • Make aura display field Version an Option since it's not always present. If not present, we will try to use the version from the URL field.
  • Fix the API calls to percent encode the parameters. Some slugs have & symbol in it, causing error in built URL

Checklist

  • Tested on Windows
  • Tested on MacOS
  • Tested on Linux
  • Added an entry to CHANGELOG.md if knowledge of this change could be valuable to users

@tarkah
Copy link
Member Author

tarkah commented Jan 4, 2021

@casperstorm so we don't actually need to account for "unkown" add-ons to fix the issue at #466. The percent encoding fix was all that was needed (the API wasn't returning inconsistent results, our URL was inconsistent with the badly encoded parameter sometimes being 1st, 2nd .. last in the list of slugs, changing the number of returned results).

Copy link
Member

@casperstorm casperstorm left a comment

Choose a reason for hiding this comment

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

Nice!

@tarkah
Copy link
Member Author

tarkah commented Jan 4, 2021

Users on both issues confirmed the fixes here resolved things for them! I'll go ahead and merge.

@tarkah tarkah merged commit 868be3c into development Jan 4, 2021
@tarkah tarkah deleted the fix/weak-auras-fixes branch January 4, 2021 23:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WeakAuras not loading/missing auras Failed to parse WeakAuras Error
2 participants
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载