Comment has been collapsed.
Comment has been collapsed.
I did used to add it inside, but that was a long time ago, so I don't think anybody is still using that version. And when I added it inside, it was on the right side. Looking at your logs, the icon is on the left side, so it couldn't be mine because when I started adding it on the left side, it was always outside of the anchor, and as a span
element. I really don't know what else could be adding ban icons though, since the Whitelist / Blacklist Indicator script adds it to the right if I remember correctly.
And I think you should always use textContent
instead of innerHTML
for grabbing text from elements.
Comment has been collapsed.
i don't use your "find those bans" nor anything that adds the ban icon next to users. but could it possibly be "Label black-/white- listed users" inside of SGT Frogs script? -- it only adds a blacklist (possibly ban icon instead?) icon next to users that you have blacklisted though, not one that finds people that have you yourself blacklisted.
just a thought.
Comment has been collapsed.
You're right that SGT Frog adds an icon inside it, but it's a fa fa-star-o
. At least here. Don't know if it can be changed.
Comment has been collapsed.
Oh I was looking at my whitelist, duh. Bingo! It's SGT Frog.
Comment has been collapsed.
you worded that just right so even those without code knowledge but just basic site/script knowledge could help. glad to of helped. and yeah i never did use that feature from that script, just remember it being there was all. i use revils version of it though because it wasn't as intrusive looking.
Comment has been collapsed.
Well, I personally like to add it to the whitelist / blacklist pages because I can see that it's highlighting every user as it should and go sync it if it's not.
Comment has been collapsed.
I meant just in general. Using textContent
to retrieve text from elements is better because it doesn't parse the text as HTML, so it has better performance. You could maybe make sure to get only the element's text by iterating over its nodes looking for a text node.
Comment has been collapsed.
I have a different rule because I try to optimize my code as best as I can based on what I know, even if the difference won't matter. :P I used to use innerHTML
as well, but then I learned about its evils and started using textContent
and insertAdjacentHTML
instead.
Comment has been collapsed.
Hope I covered all bugs. ^^
Comment has been collapsed.
This feature is missing for me since update 2.8.something. I use another script to reorganize the screen (I think ESG do that) and no check button appears on my screen. Why don't you insert this feature as popup since it will work with only one request?
Comment has been collapsed.
It should be back with v2.9.3. Are you using the latest version? And is it enabled in the settings menu? I don't know what you mean by popup, but that wouldn't prevent multiple requests from being made at all.
Comment has been collapsed.
It is back. I just updated... Thanks. :P
By the way, the popup I mentioned is like the one that appears to show the users when the scan is done.
I don't know how you guys call that box that I identify as popup. :P
Comment has been collapsed.
Just for you to know, it says that I'm already performing a check over than 2 hours...
It keeps saying that even when I closed Chrome and restart my PC. At this point I really think the check thing is stuck.
Comment has been collapsed.
I knew that was premature. :P
Comment has been collapsed.
Comment has been collapsed.
Comment has been collapsed.
dra-mat-ic-a-lly slower.. lol. currently on user 85 of 104 users in a scan started after this update. (approx 1hr ago)..
i think it maybe able to get bumped up a hair safely, but i understand keeping it down there till you here from cg. (if he ever replies anyways) xD
Comment has been collapsed.
Well, I can change how many requests are made per second at any point, so maybe I'll increase it to 2 or 3, since Not Received Finder already takes 1 and Sighery was told by knsys to keep it at most at 5.
Comment has been collapsed.
2 i would think is safe leaving that little bit of wiggle room, but is it per tab based or across all tabs it would still be limited? if multiple tabs can be doing multiple scans, that is more likely what was causing the 700page request in a minute that cg was talking about really.. so keeping it this limited maybe better if people could open 20 tabs making it 20 requests a second again.
edit: scan completed and caching worked great, refreshed and told it to scan again. it only scanned users and instantly loaded the results.. ^^
Comment has been collapsed.
Increased it to 2 per second. You cannot perform it in multiple tabs, is it not blocking you if you try?
Glad to know it's working. ^^
Comment has been collapsed.
nice on increase ^^
oh, yes it certainly is blocking me now.. it wasn't before cg said anything, but that's good except.... refreshing the page while in the middle of the scan won't allow me to start a scan anymore even after exiting the browser and returning.
Comment has been collapsed.
I'm currently trying to see if I can come up with a way to solve this, since the checker only frees the next check at the end of the current one, so stopping it before letting it finish will never free it.
Comment has been collapsed.
yeah i tested to see that message and then thought maybe i might see if i could trick it by opening up the tabs of stuff i wanted to scan before i started any scan and tell them to all check that way without triggering it.. i thought refreshing the page would just kill the scan, so stuck like that.. ^^
Comment has been collapsed.
Comment has been collapsed.
Comment has been collapsed.
Comment has been collapsed.
Comment has been collapsed.
Comment has been collapsed.
Hi, I was on WL/Group GA page and I tried to make WL Check. I think the checkbox is still quite confusing. I understood it like this: unchecked - it checks only, for example, page 1, or page 2, or page 3, and so on (so it means 10 pages for me) and checked - it checks all pages ON the WL/Group pages. But it seems I was wrong, because if it is checked, it seems to check all steamgifts - it means hundred of pages, right?
I think it is not good. It should work like I thought it worked. I had to finish it after scanning several hundred of pages, because it would have taken forever to check all SG.
Well, it was yesterday and today I still cannot perform a check, because it says I am checking now. Is it really performing the check on the background? Can I stop it somehow?
Comment has been collapsed.
Nope, it only checks all the pages of the current list. Something might have gone wrong. It kept showing "Retrieving users (page x)..."?
No, it's just that prior to v2.9.6.1, if you interrupted a check it wouldn't be able to know that and so it wouldn't free the checker. I just pushed an update that should fix that for people that did that before v2.9.6.1.
Comment has been collapsed.
It's saying "Retrieving users (page x)..." forever?
Comment has been collapsed.
It looks like that it doesn't work on default group/wl page like this:
https://www.steamgifts.com/giveaways/search?type=group
Or pages whose are not the last.
For example, https://www.steamgifts.com/giveaways/search?page=9&type=group
It scans hundreds or thousands pages.
But it works well when I am on the last page in group/wl like this:
https://www.steamgifts.com/giveaways/search?page=10&type=group
It scans the 10 pages I have in the group/wl page.
Edited.
Comment has been collapsed.
It works for me without any problem, though my list is pretty short because of hidden giveaways, only 20 GA - 1 page. Can it be then that something is up with pagination?
Edit: more than one page of all giveaways - it also works properly. So something up with groups / additional filters?
Comment has been collapsed.
Actually now it should, there was still an issue before.
Comment has been collapsed.
I have a feeling I know what it is. If you change search?page=9&type=group
to type=group&search?page=9
does it work?
Edit: Nevermind, that's broken. I'm trying to see if I can fix it.
Comment has been collapsed.
Comment has been collapsed.
Well, not every single person. It lets you check a certain list or page(s). And only if the person has already made a giveaway. :P
Comment has been collapsed.
Right now it disables only on /giveaways/
, not on /giveaway/
, since the later can be used for lists of entries / winners.
Comment has been collapsed.
back to this again, with the settings of both of them off in the menu like photo #1 you get photo #2 xD hehe
you shouldn't be able to even disable the rule checker in /giveaway/ (or at least not on the winners pages), but it is /discussions/ & /giveaways/ is where i was hoping to disable the rule checker.
2nd screenshot is from /giveaway/abcde/gamename/winners page
edit: better clarified (hopefully)
Comment has been collapsed.
Comment has been collapsed.
Feature request:
Simple version - I would like to have the possibility to automatically send keys to the winner(s) of an ended giveaway when I click on the Unsent link. This would be enough for me.
Better version - there would be also a button that would send all keys to all winner in all ended giveaways.
Advanced version - there could be lots of improvements. For example, one could set it to send keys just to whitelisted users or users who already had won and marked the game as received. Also it could exclude giveaways that have unsent keys because the GA is waiting for a reroll. It could exclude also users where the script detected not received games.
Comment has been collapsed.
Simple and better versions sound doable, not sure about the advanced one. I'll look into it.
Comment has been collapsed.
This has been added, in case you missed it. :) Well, the better version at least. I'll add the ability to send them while clicking on Unsent later. And how is it possible to know if a giveaway is waiting for reroll?
Comment has been collapsed.
Very good, thanks! I will test it when some giveaways of mine end. I don't know if it is possible to check if a GA is waiting for reroll I had just the idea. Or maybe the script could save the info about reroll after the user clicks on the reroll link?
Comment has been collapsed.
Added this. If you submit a reroll request after v4.2.1, it will not send the gift to that user.
Comment has been collapsed.
if you're taking on request i got one i'd love to see. ;)
better rule checker. a button on the giveaways winner page (and even user profile pages too) that checks if they have any "not received" or "multiple wins for same title" -- suggested here
one that doesn't have to open up in 2nd & 3rd tabs.. basically similar to the wl/bl checker.. click the button and it heads over to the appropriate sgtools rule checks pages, if both pass, it comes back with a green checkbox or something showing it is safe to send the gift. or if they fail then it would come back with a red X that is clickable to show the details of why they failed the rulecheck..
then to extend it a little better, you could add in columns for multi-copy giveaways, that way on a 20copy giveaway you don't have to open up 40tabs with all the diff rule-checking.. you can simply press the button to check all users, and send to all the people who passed instantly, then do investigative work on those that fail to determine if the game should be sent or if a support ticket should be filed.
edit: oh and groups pages too that other user wanted the rulecheck to work on, it could technically scan all the same places that the wl/bl scanner checks. this could kinda tie in with pildas suggestion too.. have it automatically send keys to people who pass the rule-check but not those that fail it.
Comment has been collapsed.
I'm currently implementing this, so it might come later today. :)
Comment has been collapsed.
i kinda figured you already planned too ^^
too good of a feature to skip out on really. xD
edit: whenever that feature shows up.. i think you'll be fully replacing sgt-frog for me.. all the features i used from that script this one is taking on.. only thing i'd lose that i don't really care about is avatar hover pop-out.
Comment has been collapsed.
Only problem is adding too many panels to the sidebar will mess with fixed elements. Maybe I'll have to add an option to toggle between panels.
Comment has been collapsed.
well, it maybe more confusing then it already is. but could it just be the same button as the wl/bl checker but with a couple more tickbox options?
Only check revilheart.
Only check rulechecks.
Also check whitelist.
Also check rulechecks.
Check all pages.
Comment has been collapsed.
Do you think I should include DLCs that are probably activated and well-known false positives that are whitelisted in the results?
Comment has been collapsed.
It might take a while because depending on the number of wins an user has the requests to SGT can be slow. Are you sure it froze? Any errors in the console?
Comment has been collapsed.
You're welcome. xD
It would be great if I could expand it to Pilda's suggestion now. :) And I could also add the same functionality of highlighting the users around SG like the WBC feature does if you want, since the results are also stored.
Comment has been collapsed.
yeah, that would be awesome. xD
either way is fine with me on highlighting the users. i certainly wouldn't disable that feature if it were added, but the checking in general was what i wanted. opening up a half a dozen tabs when several GAs ended was a pain. i don't mind if it only does one GA at a time if i only have to bother with one tab at a time. ^^
edit: actually the highlighting would come in handy on those really large copy GAs (but not quite 50 where it auto-sent keys anyways)
otherwise i'd be hitting the scan button over and over cause i wouldnt be able to remember results for 15+ people. ^^
Comment has been collapsed.
I'll add it then. I'm just concerned now that there might be way too many icons in front of each user. :P
Comment has been collapsed.
There is. That actually sounds like a better idea than adding yet another icon.
Comment has been collapsed.
I'll just use the same colors as the green / red from the received / not received class. That way it will be compatible with all themes.
Comment has been collapsed.
Comment has been collapsed.
Comment has been collapsed.
caching doesn't seem to work as well now (idk which update goofed it though). run a scan on a small group till it completes with results, refresh page to scan it again, and instead of just doing a quick scan of the users and popping up with results again it does the full scan all over again.
Comment has been collapsed.
If it's a small list (with one page only) or single user, yes, because it doesn't need to retrieve any pages. ^^
Comment has been collapsed.
Comment has been collapsed.
44 Comments - Last post 21 minutes ago by Chris76de
16,292 Comments - Last post 56 minutes ago by IAMERROR404
1,518 Comments - Last post 2 hours ago by ayuinaba
517 Comments - Last post 4 hours ago by Marius11
372 Comments - Last post 4 hours ago by Marius11
449 Comments - Last post 4 hours ago by Marius11
55 Comments - Last post 6 hours ago by XfinityX
198 Comments - Last post 3 minutes ago by samwise84
34 Comments - Last post 13 minutes ago by amusedmonkey
803 Comments - Last post 14 minutes ago by Mhol1071
106 Comments - Last post 20 minutes ago by Steamgifty
40 Comments - Last post 23 minutes ago by Fluffster
9 Comments - Last post 33 minutes ago by MayoSlice
3,365 Comments - Last post 36 minutes ago by Svarion
Enhanced SteamGifts & SteamTrades (ESGST)
An extension / userscript that adds some cool features to SteamGifts and SteamTrades.
If you find any bugs or have any feature requests, please file an issue here.
Compatibility
Tested and confirmed as working:
Not tested but should be working:
Not supported:
Installation
There are two different ways to use ESGST: extension or userscript. You can choose which one you want to use, but the extension is a lot faster and has a better peformance than the userscript, so I recommend it, although there is no major difference between them, except for a few options that are only available in the extension because of limitations in the userscript.
Option 1 - Extension
It should be possible to install the extension in any Chromium-based browser from the Chrome store (such as Opera, Vivaldi, etc...), any Firefox-based browser from the Firefox store (such as Waterfox, etc...), and any browser that uses the Phoebus system from the Pale Moon store (such as Basilik, etc...).
You can also use the extension on Android through Firefox for Android, but some features might not work or not be optimized enough for mobile.
Option 2 - Userscript
To install the userscript, you must install Violentmonkey, Tampermonkey, Greasemonkey or some other userscript manager first. Then click here and you should be prompted to install it. I recommend Violentmonkey or Tampermonkey, because the userscript uses the GM_addValueChangeListener API to communicate changes in the storage between tabs, and that API is not available on Greasemonkey, so a polyfill is used for Greasemonkey, which is not very effective and can affect the performance.
You can also use the userscript on Android through Firefox with Violentmonkey or USI, but some features might not work or not be optimized enough for mobile.
Beta Versions
The latest beta version for both the extension and the userscript is always available in the latest pre-release in the releases page.
Comment has been collapsed.