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

[MBL-1816] PLOT Ineligible state #2220

Open
wants to merge 6 commits into
base: jluna/MBL-1815/plot-plan-selector-selected-state
Choose a base branch
from

Conversation

jovaniks
Copy link
Contributor

📲 What

This PR implements the "PLOT ineligible" state for the Payment Plan Selector. It disables the "Pledge Over Time" option when the total pledge amount is below the threshold of $150.00 and displays a note indicating the ineligibility, with the amount converted using the project’s currency.

🤔 Why

The "PLOT is NOT eligible" state ensures backers understand why the "Pledge Over Time" option is unavailable. This limitation is based on the total pledge amount being below the required threshold of $150.00. By providing clear feedback, users are informed of the eligibility requirements.

🛠 How

Adding the new field ineligible to PledgePaymentPlanOptionData and PledgePaymentPlansAndSelectionData

👀 See

Eligible Ineligible
Simulator Screen Recording - iPhone SE (3rd generation) - 2024-12-11 at 11 14 35 Simulator Screen Recording - iPhone SE (3rd generation) - 2024-12-11 at 11 13 33

✅ Acceptance criteria

  • Tests should pass
  • Ineligible UI should show when total < 150
  • Eligible UI should show when total >= 150

⏰ TODO

  • Connect the API to have the ineligible flag from BE

…jluna/MBL-1816/plot-ineligible-state

# Conflicts:
#	Kickstarter-iOS/Features/PledgeOverTime/Views/PledgePaymentPlanOptionView.swift
#	Library/ViewModels/NoShippingPledgeViewModel.swift
#	Library/ViewModels/PledgePaymentPlansViewModel.swift
@jovaniks jovaniks self-assigned this Dec 11, 2024
@jovaniks jovaniks changed the base branch from main to jluna/MBL-1815/plot-plan-selector-selected-state December 11, 2024 20:58
Copy link
Contributor

@scottkicks scottkicks left a comment

Choose a reason for hiding this comment

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

Nice! Had just a few suggestions that should be pretty easy to get in before we merge this

@@ -174,6 +205,7 @@ final class PledgePaymentPlanOptionView: UIView {
self.titleLabel.rac.text = self.viewModel.outputs.titleText

self.subtitleLabel.rac.text = self.viewModel.outputs.subtitleText
self.subtitleLabel.isHidden = true
Copy link
Contributor

Choose a reason for hiding this comment

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

This can probably go in applySubtitleLabelStyle with the other styles.

Library/Extensions/UIView+Helper.swift Show resolved Hide resolved
…jluna/MBL-1816/plot-ineligible-state

# Conflicts:
#	Library/ViewModels/NoShippingPledgeViewModel.swift
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