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

Added refresh function #18

Open
wants to merge 1 commit into
base: release
Choose a base branch
from
Open

Added refresh function #18

wants to merge 1 commit into from

Conversation

apedley
Copy link

@apedley apedley commented Sep 25, 2022

Added a function to refresh current items

@mlsof21
Copy link
Owner

mlsof21 commented Sep 26, 2022

This could be useful, but DIM already has an auto-refresh that happens (though I think it's been off since KF raid launch) in the background. I've actually logged an issue related to switching characters in-game then attempting to move an item via voice. It will usually fail until you either reload the page or hit the manual refresh (circling arrows in top right). I think the refresh itself could be useful if we can detect that a transfer/equip/store should have happened but didn't; however, I think it's overkill as a command.

@apedley
Copy link
Author

apedley commented Sep 26, 2022

Since the KF launch and the API issues, I find myself having to refresh manually before doing what I would want to use the extension for which I feel defeats the purpose. Other times I want to get the refresh started because I know I'm about to go through the loot I just got but that was more of an issue when it could take up to a minute.

I guess I don't see how this is overkill but turning farming mode on and off and equipping the max power loadout isn't.

@delphiactual
Copy link
Contributor

Because auto-refresh is off right now at the behest of Bungie. It should be back on soon.

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.

None yet

3 participants