pfh59 / eve-whmapper

simple eve online wormhole mapper.
MIT License
24 stars 7 forks source link

[Support] Dev/Test Environment #173

Closed rdymade closed 10 months ago

rdymade commented 10 months ago

Hello!

I don't know if you've already seen my post on the forums @pfh59 but here i am (ReadYY).

I've already analyzed the code a bit to understand what pieces are for what and how they interact.

I would need some support setting up a proper testing environment in order to make changes and improvements.

Would be much appreciated if you could give me a lead.

pfh59 commented 10 months ago

Hi @rdymade,

I'm using visual studio code and docker on my Mac , but you can use it on windows.

I use docker to run posgresql container via command 👍 docker run --name db -e POSTGRES_USER=postgres -e POSTGRES_PASSWORD=secret -e POSTGRES_DB=whmapper -p 5432:5432 -d postgres

Then checkout code, the better way is to fork the repo. Don't forget to create your API key to https://developers.eveonline.com (see : https://github.com/pfh59/eve-whmapper#register-your-app-with-ccp)

Edit appsettings.json in WHMapper and WHMapper.tests

Compile and launch WHMapper server on local machine

rdymade commented 10 months ago

Thank you! i got it up and running just fine already doing some dev (will for sure do it via a fork) but right now i'm just fiddling around.

First thing i saw is that you already implemented the paste signatures on the default view feature but its somewhat buggy i noticed that it works just fine when i first select a system and then click into the top bar (the darker gray one where "Wormhole Mapper" is written so my assumption is that Blazor intercepts the CTRL + V somehow before the WHMapper.lib.module.js can do its job.

Secondly i'm fiddling on a feature for presistently adding a system status (unknown/friendly/occupied/hostile) with matching border colors as they are in pathfinder.

edit: There's two ways i think we could handle this 1st keep the WHSystem in the DB and store information like this there or store this information seperatly in the WHNote what would you suggest? Also how did you do DB Migrations? are those scripts just written manually or somehow generated?

pfh59 commented 10 months ago

hey,

I know the problem of copy/pasting signatures and the behaviour seems to be different depending on whether you use chrome or firefox.... For me, the Blazor-Diagram component has a strange behaviour with copy/paste. Can you open a bug about this and I'll try to fix it?

As far as functionality is concerned, you can add one note per system with a maximum of 255 characters. I can extend it and add more information. I will generate database migrations with the dotnet ef migrations command. Can you create another problem as a feature?

If you want to help me develop, I accept pull request contributions only. After checking your code and the build continues to be green, you can merge it. Otherwise, I can do it, but I work on it 2/3 evenings a week.

rdymade commented 10 months ago

going to create some issues then. i'll write it in the issue when i take over development for them. and will provide them via pull request ofc! ;)