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

Update steam-auto-queue.user.js #2

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Update steam-auto-queue.user.js #2

wants to merge 3 commits into from

Conversation

Kasp42
Copy link

@Kasp42 Kasp42 commented Jun 24, 2016

Add skip agecheck

@FiveElementNinja
Copy link
Owner

Hi @Kasp42, thanks for the update! Would you mind submitting this to the development branch instead if it's not too much trouble? I like to do testing in that branch before merging into the master branch. Part of the reason for this is because I use several other scripts on Steam and I like to make sure nothing conflicts or causes problems. For example, I use Enhanced Steam, and one of the features it has is to automatically skip the age check that you added code for. It shouldn't cause a problem in this case, but you just never know what someone else's code might be doing.

Also, is it necessary to randomly choose the month, day, and year? Would it be better to simply select January 1, 1900 (or whatever the lowest year is)? That's what I was planning on doing anyway in the next update. I'd prefer it to do that, but I'm open to suggestions.

Thanks again for the contribution!

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