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

Freemium PIR: Add FreemiumPIRFeature Type and Add New State Property #3160

Merged

Conversation

aataraxiaa
Copy link
Contributor

@aataraxiaa aataraxiaa commented Aug 27, 2024

Task/Issue URL: https://app.asana.com/0/0/1208098668401283/f

Description: This PR targets a feature branch, and is just the first part of a larger piece of work. This PR adds:

  1. A new FreemiumPIRFeature type, which is used to determine if Freemium PIR is available
  2. A new property - isCurrentUser to FreemiumPIRState indicating the user is a current freemium PIR user

Steps to test this PR:

  1. None, Ensure CI is green

Definition of Done:

Internal references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

@aataraxiaa
Copy link
Contributor Author

Updated @miasma13.

@aataraxiaa aataraxiaa requested a review from miasma13 August 27, 2024 16:04
@aataraxiaa
Copy link
Contributor Author

Updated based on PR feedback and MM discussion @miasma13 . Thanks for being so thorough, it’s awesome! 👍🏼

Copy link
Contributor

github-actions bot commented Aug 28, 2024

Warnings
⚠️ PR has more than 500 lines of code changing. Consider splitting into smaller PRs if possible.

Generated by 🚫 dangerJS against 020a4f7

Copy link
Contributor

@miasma13 miasma13 left a comment

Choose a reason for hiding this comment

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

Looks great, clean and simple 👍

@aataraxiaa aataraxiaa merged commit 8da7069 into pete/feature/pir-freemium Aug 28, 2024
21 of 22 checks passed
@aataraxiaa aataraxiaa deleted the pete/feature/pir-freemium-feature-setup branch August 28, 2024 11:38
aataraxiaa added a commit that referenced this pull request Sep 10, 2024
…3160)

Task/Issue URL:  https://app.asana.com/0/0/1208098668401283/f

**Description**: This PR targets a feature branch, and is just the first
part of a larger piece of work. This PR adds:
aataraxiaa added a commit that referenced this pull request Sep 18, 2024
…3160)

Task/Issue URL:  https://app.asana.com/0/0/1208098668401283/f

**Description**: This PR targets a feature branch, and is just the first
part of a larger piece of work. This PR adds:
aataraxiaa added a commit that referenced this pull request Oct 8, 2024
…3160)

Task/Issue URL:  https://app.asana.com/0/0/1208098668401283/f

**Description**: This PR targets a feature branch, and is just the first
part of a larger piece of work. This PR adds:
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