Skip to content

Update iOS project version and add recreation activities menu item#84

Merged
RDMurray merged 6 commits into
soundscape-community:mainfrom
RDMurray:activities
Mar 19, 2024
Merged

Update iOS project version and add recreation activities menu item#84
RDMurray merged 6 commits into
soundscape-community:mainfrom
RDMurray:activities

Conversation

@RDMurray

@RDMurray RDMurray commented Feb 8, 2024

Copy link
Copy Markdown
Contributor

This is hopefully a start to getting the app working with recreational activities again. currently build 15 on testflight but doesn't work yet.

@steinbro

Copy link
Copy Markdown
Member

I take it TestFlight build 16 came from this branch?

@steinbro steinbro mentioned this pull request Feb 10, 2024
@RDMurray

Copy link
Copy Markdown
Contributor Author

@steinbro yes. I'm going to change this to draft because it is not ready to merge.

@RDMurray
RDMurray marked this pull request as draft February 10, 2024 15:09
@RDMurray

Copy link
Copy Markdown
Contributor Author

I'll rebase this once the issue with #73 is resolved.

@RDMurray

Copy link
Copy Markdown
Contributor Author

I found the issue in #73 and pushed it to main. This branch has been rebased.

@RDMurray
RDMurray marked this pull request as ready for review March 14, 2024 21:20
@RDMurray

Copy link
Copy Markdown
Contributor Author

This is included in build 19 on testflight. the new geojson code is not on testflight yet.


// MARK: Test GPX Parsing

static let baseURL = URL(string: "https://example.com")!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm guessing this is extraneous, though probably harmless.

Comment on lines +57 to +59
default:
// no other versions currently supported
break

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm curious why there wasn't a default case before. I'm not sure what Swift semantics are for unhandled cases, but the behavior with the default case (return an empty URL) seems correct.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think all cases were handled explicitly before. Swift does require all cases to be handled either explicitly or with a default case.

I was thinking of proposing a v4 link in the future that would contain the full URL to the activity.gpx file. That would allow the authoring tool to be used on other domains for development etc.

@RDMurray
RDMurray merged commit 23f59a8 into soundscape-community:main Mar 19, 2024
@RDMurray
RDMurray deleted the activities branch March 19, 2024 23:54
timxie140 pushed a commit to timxie140/soundscape that referenced this pull request Apr 4, 2025
…oundscape-community#84)

* iOS: Re-enable activities and update URLs

* Fix the URL for downloading activities

* Support relative links for images and audio files in activities.

* bump build to 17

* Fix audio clips in activities, remove landscape from orientations. (build 18)

* Fix audio and image loading for activities, update UnitTests
bumped build number to 19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants