EVE-University / whctools

WHC app for joining
MIT License
1 stars 1 forks source link

Alt marked accepted without application #11

Closed mome-borogove closed 2 weeks ago

mome-borogove commented 1 month ago

I don't know exactly the steps leading up to this, but the situation ended up as:

I don't believe this should be possible. In fact, I'm not even sure why the main was originally listed as on cooldown due to skills, seeing as they'd been removed from the community.

Timestamps for checking logs:

It's unclear to me what exactly went wrong here, but something did. If alt4 was actually a member at 9:02pm, then the user panel was incorrectly showing it as a non-member. If alt4 was not a member at 9:02pm, then it was incorrectly accepted as a member without submitting an application after the main was accepted. Also, regardless, there should be no way that all characters were removed from the Community and then alt4 was accepted without a new application.

lynkfox commented 2 weeks ago

i think this is related to the long load period that was caused by trying to gather all the skills for all the apps all at the same time.

Ive moved the skill check window data to its own function that is only called when the button is pressed, so on the assumption that that lag was causing additional issues such as this, going to put a hold ont his issue until the new version is out and we can see if it continues

mome-borogove commented 2 weeks ago

Cannot repro this. Given the behavior of v0.2.4, I'm inclined to think that it's not possible to produce anymore. And I tried.

Considering it closed. Will reopen if we find a way to trigger it again.