Skip to content
This repository was archived by the owner on Mar 17, 2024. It is now read-only.

Startup items fix#258

Merged
MaxXor merged 8 commits intoquasar:masterfrom
yankejustin:StartupItemsFix
Jun 2, 2015
Merged

Startup items fix#258
MaxXor merged 8 commits intoquasar:masterfrom
yankejustin:StartupItemsFix

Conversation

@yankejustin
Copy link
Copy Markdown
Contributor

Fixes Issue 256.
I have no time to post a better description, so I apologize. I need to clean the method that removes registry key items later. :)

Added RegistryKey Extensions/Helper methods. Cleaned up and improved
HandleGetStartupItems.
Added some more documentation for the GetFormattedKeyValues extension
method.
If the client cannot give us anything, don't crash over it. :)
We can't use these entries so just leave.
This makes the code more readable and the intent more obvious.
For those that like to see the type of exception thrown, this makes it
more clear and meaningful.
MaxXor added a commit that referenced this pull request Jun 2, 2015
@MaxXor MaxXor merged commit a12db99 into quasar:master Jun 2, 2015
@MaxXor
Copy link
Copy Markdown
Contributor

MaxXor commented Jun 2, 2015

Added it. No need to worry about.

@yankejustin yankejustin deleted the StartupItemsFix branch June 2, 2015 20:08
@MaxXor
Copy link
Copy Markdown
Contributor

MaxXor commented Jun 3, 2015

I have to admit, these changes didn't even work after I've fixed them. :(

@DragonzMaster
Copy link
Copy Markdown
Contributor

The problem that startup manger only works on my PC @MaxXor :(

@MaxXor
Copy link
Copy Markdown
Contributor

MaxXor commented Jun 3, 2015

Use the latest commit.

@DragonzMaster
Copy link
Copy Markdown
Contributor

@MaxXor
Unfortunately it seems that I still have this problem even after using latest commit (616)
xr3

@yankejustin
Copy link
Copy Markdown
Contributor Author

Start the client in debug and see the exception that is thrown please.

@DragonzMaster
Copy link
Copy Markdown
Contributor

It is working good on my pc but when I send it (client) to someone it doesn't work as shown in the picture
So do you mean that I have to send debug client instead of release one ??
@yankejustin

@MaxXor
Copy link
Copy Markdown
Contributor

MaxXor commented Jun 3, 2015

no, he doesn't mean this. Please update your clients to the latest version please. :) And it will show you the error, please tell me this.

@MaxXor
Copy link
Copy Markdown
Contributor

MaxXor commented Jun 3, 2015

Please write in your #256.

@quasar quasar locked and limited conversation to collaborators Jun 3, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants