No station list #189
Closed
No station list #189
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Clean up credit dialog - change icon to access about - replace description - replace credits
Still trying to use SwiftRadio. Fixed.
- Added fastlane - added fastlane config for snapshots - Changed UI tests to take appropriate snapshots
- screenshots dir - report.xml in fastlane
Turn tests back on now that they work
UI tests and fastlane setup
- Added labels to stack - added code to set PLAYING LIVE label
- This version hits the /broadcast endpoint and tries to parse the current_show. If there is none, then we assume it's Spud. This avoids the complicated decoding of a dual-typed field (current_show, which is either false or a struct). - Updates the UI to show the current DJ's name.
…igh enough - was >= 180; changed to <= 300
- run on the iPad Pro now seems to work, and the "constraint broken" errors are fixed.
Apple required update
- bump build # to 21 - UI tweaks so presentation looks good on iPad too - Updated RadioAPI to properly parse live shows (note that we still need to properly handle when there's no live show with enum parsing, and look into whether there are multiple fields we'll have to deal with this in) - DJ field and LIVE field now added and tested - Remove stationDesc label; extraneous for a single station This might be sufficient for the 2.0 release.
- latest, website, route, and feed in Show - ID, encore, and override in BroadcastShow
Remove fields not showing up when a show is on: - latest, website, route, and feed in Show - ID and encore in BroadcastShow Update UI - DJ field is name only - if a live DJ is on, show a mic icon - Iif performing live ([LIVE] at end), show icon and LIVE
The live check now downcases the string and then looks for :
- [live]
- {live}
- (live)
- <live>
- «live»
I will add other bracketing styles if someone uses them.
We only do the live check if we have detected a live DJ as well.
Lots of tracks in the library use this, don't want a false positive.
Add IDEPreferLogStreaming=YES" CLI argument to scheme
- Force the stations list controller to immediately switch to the first station in the list - Have the now-playing controller do the following - hide the back button - when first loaded, stop the player Cleaned up the info view so the default station is RadioSpiral. Fixed a bug that would cause the info page to crash the app.
- update build # and version - remove RadioAPI debugging - make DJ name and state indicator one line in preparation for full track-release-artist parsing - Add "LiIVE on RadioSpiral because I'm never going to get 7 DJs to cooperate on marking themselves as playing live
joemcmahon
force-pushed
the
no-station-list
branch
from
69f68bb to
2875b59
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Gets rid of the interstitial page. Prepping for adding release name to displays and sharing.