-
Notifications
You must be signed in to change notification settings - Fork 172
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
. #94
Open
Helothane
wants to merge
268
commits into
evemondevteam:master
Choose a base branch
from
peterhaneve:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
. #94
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Parse wallet amount properly on systems with non "." decimal local
Project References to .NET 4.6.1 and TimeSync hang fix
…than once, which could cause crashes and duplicate notifications. - Mostly fixed EVE notifications. - If no ESI developer info is entered, prompt automatically goes to the right settings page. Still need a link to instructions. - Installer updated. Version numbers rolled to 4.0.0 series. Added a warning to the installer to clear settings from older versions.
…ng bug with the new character sheet approach. - Fixed errors caused by lack of corporation roles, which fixed personal market orders - Fixed upcoming calendar events, only to discover that they have no renderer anyways - Still issues with contracts
…ter contracts and items hopefully for good.
…ESI key. Fixed an issue where implant sets would have blank names.
… issues, including missing solar systems. Moved URL to new ESI location which fixes some latency issues and may fix "trust failures"
…int if Hammer does not know about the structure (needs to be set in EveMon and EveMon.Common)
…menu option is doing (add/manage characters)
Changed the old API Key labels to more accurately represent what the …
…uppressed some duplicate skill trained notifications.
…itoken Removed preprocessor guarding Citadel Info api priority changed from Fallback from ESI strucutre call to Hammerapi (Hammerapi is still used if no token is provided) Tokens are now resolved in IDToObjectProvider.LookupID
…ets accounted for in QueueID already
…y monitors have completed
…s still in queue). This needs to be fixed in some other way.
…SI standard. Skills list should now show the correct level/sp after both Skills and SkillQueue completed, no matter which order. Excess skill completed notifications should also be prevented now (as skills aren't imported at lower than actual level). One thing remains - skill queue import only will import at wrong levels.
Completed skills are leveled up, skills still training get current SP updated If any skill queue modifications were done, trigger SkillQueueUpdated again (fixes issue with SkillQueueControl not being invalidated after completed skills are removed)
…t before the event
Updates to fix changed value for reactions (for tracking reaction jobs)
Prepare 4.0.14 release.
…if displayed. Fix issue #187 until CCP provides new flags update by rejecting invalid flag names.
…fter ESI change removed market orders which were completed or expired. Fixes #196.
Fix issue #188 by allowing notifications to be closed with right click.
Should fix #204 by updating the SMTP server for live/hotmail accounts.
…g on skill level details
Fix #205 by fetching historical market orders along with normal orders. Prepare 4.0.16 release.
As of https://www.eveonline.com/article/patch-notes-for-september-2017-release Repair Systems is no longer in the starting skill set. Delete it. By experiment and despite the comment in those patch notes saying that small empire-specific turret skills would be removed, new players do get those. Add them. Likewise new players get their empire-specific frigate skill at 1. Add that. Per https://wiki.eveuniversity.org/Starting_skills and confirmed by experiment, Caldari get the three starting shield skills at 2 (instead of 1 like the other empires) and the two starting armor skills at 2 (instead of 3 like the other empires). Also, Minmatar get Controller Bursts at 1 (instead of 2 like the others, and despite not getting anything else bumped to make up for it...#inrustwetrust). Add those six skills to the race skill dictionaries accordingly. By experiment, for all empires, targeting skills all got a 1-level bump, and Marketing 1 got added to the trade skills. Can't find a patch note for this and not shown on EVE-Uni yet (I will update there) but tested in-game. As of https://www.eveonline.com/article/pxf1u0/patch-notes-for-september-2019-release Thermodynamics is no longer included in starting skills. This was already commented out. Just delete. Side note, Caldari new players start with 10k less SP, and Minmatar 2k less SP, than Gallente/Amarr.
…iance names where possible, to do what is possible to fix #227.
Update blank character starting skills for April 2020
SkillPlanner: highlight Omega-only skills in plan
Prepare 4.0.18 release.
… dump) since there is no ESI endpoint and the hardcoded list was getting outdated nearly monthly.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
ignore