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

[EP-401] Fix properties (phase one) #1414

Merged
merged 13 commits into from
Mar 31, 2021
Merged

[EP-401] Fix properties (phase one) #1414

merged 13 commits into from
Mar 31, 2021

Conversation

singhhari
Copy link
Contributor

📲 What

Fixing properties after phase one of Segment QA. Some of these are smaller changes like changing the type whereas some are updates to where we get the values from.

🤔 Why

Updates after receiving QA feedback for phase one of our Segment implementation.

@codecov
Copy link

codecov bot commented Mar 30, 2021

Codecov Report

Merging #1414 (f1092e2) into master (1de00c5) will increase coverage by 0.01%.
The diff coverage is 95.76%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1414      +/-   ##
==========================================
+ Coverage   86.10%   86.11%   +0.01%     
==========================================
  Files        1106     1106              
  Lines       97850    97950     +100     
==========================================
+ Hits        84251    84347      +96     
- Misses      13599    13603       +4     
Impacted Files Coverage Δ
KsApi/models/PaymentType.swift 0.00% <0.00%> (ø)
Library/UIDeviceType.swift 62.50% <ø> (ø)
Library/SharedFunctions.swift 89.05% <100.00%> (ø)
Library/SharedFunctionsTests.swift 100.00% <100.00%> (ø)
Library/Tracking/KSRAnalytics.swift 85.32% <100.00%> (+0.53%) ⬆️
Library/Tracking/KSRAnalyticsTests.swift 100.00% <100.00%> (ø)
Library/ViewModels/LoginToutViewModelTests.swift 100.00% <100.00%> (ø)
Library/ViewModels/PledgeViewModelTests.swift 100.00% <100.00%> (ø)
...ary/ViewModels/ProjectPamphletViewModelTests.swift 100.00% <100.00%> (ø)
Library/ViewModels/ThanksViewModelTests.swift 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1de00c5...f1092e2. Read the comment docs.

KsApi/models/PaymentType.swift Show resolved Hide resolved
Library/Tracking/KSRAnalytics.swift Show resolved Hide resolved
Library/Tracking/KSRAnalytics.swift Show resolved Hide resolved
Library/Tracking/KSRAnalytics.swift Show resolved Hide resolved
Library/Tracking/KSRAnalytics.swift Show resolved Hide resolved
@singhhari singhhari merged commit f6e63b3 into master Mar 31, 2021
@singhhari singhhari deleted the EP-401-fix-properties branch March 31, 2021 01:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants