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

0.69 (last one for today) #23

Merged
merged 2 commits into from
Sep 20, 2012
Merged

0.69 (last one for today) #23

merged 2 commits into from
Sep 20, 2012

Conversation

spacke
Copy link

@spacke spacke commented Sep 20, 2012

Added the amount of matches in brackets to the drop-down menu.

@njs50
Copy link
Owner

njs50 commented Sep 20, 2012

ah, i wasn't doing that because it'll break the analytics :p

i should probably fix them anyways i guess.

On Thu, Sep 20, 2012 at 3:10 PM, spacke notifications@github.com wrote:

Added the amount of matches in brackets to the drop-down menu.

You can merge this Pull Request by running:

git pull https://github.com/spacke/poe_ext master

Or view, comment on, or merge it at:

#23
Commit Summary

  • Replaced the wrong appid.
  • Added the amount of matches in brackets to the drop-down menu.

File Changes

  • M poe_ext.crx (0)
  • M poe_ext/main.js (2)
  • M poe_updates.xml (2)

Patch Links

@njs50 njs50 merged commit ddc7ca5 into njs50:master Sep 20, 2012
@njs50
Copy link
Owner

njs50 commented Sep 20, 2012

if you could pull the change to track.js back into your branch that'd be
grand.

btw: if you want access to the analytics data just send me an email address
for a google account and i'll hook you up.

interesting to see what things people actually use.

On Thu, Sep 20, 2012 at 3:13 PM, Nicholas J Shearer <
nick@simianengineering.com> wrote:

ah, i wasn't doing that because it'll break the analytics :p

i should probably fix them anyways i guess.

On Thu, Sep 20, 2012 at 3:10 PM, spacke notifications@github.com wrote:

Added the amount of matches in brackets to the drop-down menu.

You can merge this Pull Request by running:

git pull https://github.com/spacke/poe_ext master

Or view, comment on, or merge it at:

#23
Commit Summary

  • Replaced the wrong appid.
  • Added the amount of matches in brackets to the drop-down menu.

File Changes

  • M poe_ext.crx (0)
  • M poe_ext/main.js (2)
  • M poe_updates.xml (2)

Patch Links

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.

2 participants