Summer of making PRs

Summer of making PRs

8 devlogs
7h 43m
•  Ship certified
Created by Hridya

Some PRs to improve SOM experience, show amount of votes cast on project when it's in voting, AI feedback etc. but they weren't merged, but 2 were (Start voting link bug fix, show time spent on each ship)

Timeline

Ship 1

1 payout of shell 82.0 shells

Hridya

about 1 month ago

Hridya Covers 8 devlogs and 7h 43m

This was a lot of work, on ai feedback, but it turns out im not a good ruby dev :( i broke prod for a bit sob, so it got reverted unfortunate.

Update attachment

Kartikey said he'll merge the ai ship feedback! yay i fixed merge conflicts & voting progress implementation on that pr

Update attachment

The voting progress changes were merged, then reverted bc appr sm didnt want them on? Really the only changes that did get merged, was the start voting link bug & the time on ship.

Update attachment

Fixed the start voting link thing, and fixed merge conflicts w rest stuff

Update attachment

Got some changes requested - mainly regarding some ruby best practices fat models skinny controllers (I think its a ruby thing?) and i moved the regenerate route to admin namespace, and also i learned git stuff (cherry pick, checkout) useful so i rebased the voting progress into another pr for easier reviewing. Attached, is how the AI feedback will look like

Update attachment

Also added voting progress (Your ship has recieved x votes out of 18 needed, and needs x more) and fixed double escaping project description during voting.

749 being the vote numbers issue, and #898 being double escaping project desc's

Update attachment

I think ship feedback thing done
Implemented
regeneration feature for admin
AI generated feedback for ships based on votes
Backfill job to generate feedback for all previous ships before this

Update attachment

so like 50% of this time was in setting up my local enviroment why is windows so hard for ruby :sob: but I added time covered on ship events, i guess it'll be easier to test next change

Update attachment