I was working with one other intern and our first project was to add Stripe integration for paying invoices. There was already a structure for other payment processors so this was easy - just copy the pattern and we’re done! …Or not. After years of minimal feedback, someone was looking at my code.
It’s the job of the reviewer to look for mistakes, but also to start a discussion about your design decisions. Some feedback I’ve gotten:
- Does that method really belong here?
- You should really be mocking this object rather than
- That isn’t how you spell String
It passed review, let’s ship it! Not so fast. Unfortunately, people actually use this stuff by clicking on buttons and things. I know, right?
There’s no dedicated QA team at FreeAgent - this responsibility is shared between the development and the support teams - so it’s up to you document the feature using Trello, then to to find someone to run through the expected behaviour and try to break it.
At first this felt like a burden (‘works on my machine!’ - Ed.), having to wait for other people before you can get your changes out. After a while though it became obvious that the best way to get someone to help you is to help them first. I got to examine parts of the application I otherwise wouldn’t have even known about, and even found a few patterns which could be used in what I was working on.
I found someone to deploy, we’re done! Well, not quite. After your changes hit production (usually alongside a few other branches that are also complete), it’s time to nervously watch for spikes in response time and any new exceptions and hope you didn’t screw up.
Now you’re done :)
So thanks to everyone at FreeAgent for an amazing summer, it was a pleasure working with you all.
Despite being interns, every effort was made to make us feel like equal members of the team. Our code was held to the same standards, we had similar responsibilities and we were working on live projects under active use.
And I’m sure if it was one of our birthdays we would have gotten an awesome cake, too!