magento-hackathon / AsyncIndex

MIT License
78 stars 37 forks source link

New mode, options, grid and updates. #4

Closed unknownnf closed 10 years ago

unknownnf commented 10 years ago

Added:

Updated:

Flyingmana commented 10 years ago

looks good so far.

I will merge this later by hand as I dont want to remove the extra block from admin view, but I will add a config so its not added by default.(we talked already about this)

In antoher note, pull requests work better when you dont use master as source. I think every commit you add to master before the PR is merged is added to this PR.

On the other side, Iam good with git, have added your repository as remote so I can keep track of your changes even if you dont do PRs. ;)

Merging needs to wait a day, as I want to test this a bit more in detail.

unknownnf commented 10 years ago

Merging by hand not needed, I rebased my fork.

That's how every PR works irrelevant of branch. I use another repository for working with this so I won't need other commits unrelated to this before the merge, can always branch out if so.

PS: It's my job to prepare the PR so you can merge it, not yours, wouldn't even bother with these if it was the other way around.