Skip to content
Snippets Groups Projects
Commit ba01da38 authored by Imran Hussain's avatar Imran Hussain
Browse files

Merge branch 'suapiv2' into 'master'

Implement the new SU API Closes #5

See merge request !25
parents 61563af0 2bf47915
No related branches found
No related tags found
2 merge requests!28Push SU-APIv2 stuff to live,!27Push SUAPIv2 code to beta
Loading
  • Developer

    I don't have time to finish reviewing this tonight I'm afraid. It's a good start, but there's enough little errors that I would suggest holding off on merging this to beta until its had a proper check through - either by myself when I get time or another set of eyes.

  • Isabel Jenkins @unreturnable ·
    Contributor

    I think a lot of this code could be laid out a bit better (Although that goes for the whole site as well), while a rather small thing compared to actual bugs in the site the highly inconsistent use of spaces throughout the code makes some parts hard to understand. It should also be split up more into blocks by a blank line in-between blocks to help with readability.

0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment