Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add Insights tab #7

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat: add Insights tab #7

wants to merge 2 commits into from

Conversation

cbaker6
Copy link
Member

@cbaker6 cbaker6 commented Apr 23, 2022

Add's InsightsView tab to the app for graphing OCKOutcomeValue's. Be sure to look at the TODO's in the comments.

Note that completing this part should remove all charts from CareViewController.

Simulator Screen Shot - iPhone 13 mini - 2022-04-23 at 18 43 37

@@ -38,8 +38,6 @@ import CareKitUI
import ResearchKit
import os.log

// swiftlint:disable type_body_length
Copy link
Member Author

Choose a reason for hiding this comment

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

You probably won't need to remove this line in your apps.

InsightsView()
.tabItem {
if selectedTab == 0 {
Image(systemName: "chart.pie.fill")
Copy link
Member Author

Choose a reason for hiding this comment

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

Image's for your tabItem can be any you want. Feel free to change any of them to images you like better.

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.

1 participant