Closed (fixed)
Project:
Media Remote
Version:
1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
18 Feb 2022 at 21:00 UTC
Updated:
12 Mar 2022 at 09:39 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
gtr18 commentedComment #3
marcoscanoThank you for contributing! I think we are almost there, I just found a few nitpicks with the code but after that we'll be good to go.
Are we using this `type` variable or is it just leftover from other formatters?
Again, do we need the type here?
Copy/paste leftover?
We don't seem to be using the type, we could just remove it here I think.
Comment #4
gtr18 commentedComment #5
gtr18 commentedComment #6
gtr18 commentedComment #8
marcoscanoLooks great, thanks for contributing!