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.

Sign up for GitHub

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

Jump to bottom

[PB-1435]: feat/update users tier on subscription change. #52

Merged
merged 6 commits into from
Apr 28, 2024
Merged

[PB-1435]: feat/update users tier on subscription change. #52

merged 6 commits into from
Apr 28, 2024

Conversation

apsantiso
Copy link
Contributor

@apsantiso apsantiso commented Feb 23, 2024 •
edited
Loading

With the implementation of the new feature limits. We need to update users's tiers based on their subscription status, in order to accomplish that, we need to use the new /update/tier endpoint that is exposed on the gateway.

Requires:

NOTE: this should NOT be merged without mapping the productIds/planIds - tiersId in paid_plans beforehand!!

@apsantiso apsantiso mentioned this pull request Feb 23, 2024
Merged
@apsantiso apsantiso changed the title feat: update users tier when they cancel or sub [PB-1435]: feat/update users tier on subscription change. Feb 23, 2024
@apsantiso apsantiso marked this pull request as ready for review February 23, 2024 11:38
@apsantiso apsantiso requested a review from sg-gs as a code owner February 23, 2024 11:38
@apsantiso apsantiso requested a review from xabg2 February 23, 2024 15:32
@apsantiso apsantiso self-assigned this Feb 23, 2024
@apsantiso apsantiso added the enhancement New feature or request label Feb 23, 2024
@sw-wayner
Copy link
Contributor

sw-wayner commented Mar 5, 2024 •
edited by inxtci
Loading

/check-preview notify=true

Tip

Preview was successfully deployed

@inxtci inxtci added the deployed label Mar 5, 2024
@internxt internxt deleted a comment from inxtci Mar 7, 2024
@internxt internxt deleted a comment from apsantiso Mar 7, 2024
@internxt internxt deleted a comment from inxtci Mar 7, 2024
@internxt internxt deleted a comment from inxtci Mar 7, 2024
@sw-wayner
Copy link
Contributor

sw-wayner commented Mar 8, 2024 •
edited by inxtci
Loading

/update-preview drive-server preview-427-f2e81c3c78c92f8377a0f3b270fed63fcecd072b imageSuffix=-dev

Note

Pull request successfully updated 🚀
Preview updated with drive-server-dev:preview-427-f2e81c3c78c92f8377a0f3b270fed63fcecd072b

@sw-wayner
Copy link
Contributor

sw-wayner commented Mar 8, 2024 •
edited by inxtci
Loading

/update-preview drive-server-wip preview-274-31558bb7d55d2271e84a32c00d03da446f0ad967 imageSuffix=-dev

Note

Pull request successfully updated 🚀
Preview updated with drive-server-wip-dev:preview-274-31558bb7d55d2271e84a32c00d03da446f0ad967

@apsantiso
Copy link
Contributor Author

apsantiso commented Apr 8, 2024 •
edited by inxtci
Loading

/update-preview drive-server-wip 20ec4607ea04b0ae2c46d9fd7bc389dd1fd2b6c2 imageSuffix=-dev

Note

Pull request successfully updated 🚀
Preview updated with drive-server-wip-dev:20ec4607ea04b0ae2c46d9fd7bc389dd1fd2b6c2

@apsantiso
Copy link
Contributor Author

apsantiso commented Apr 8, 2024 •
edited by inxtci
Loading

/update-preview drive-server-wip preview-274-c2c3a96e3ef400219b757266e3dd352be29f2874 imageSuffix=-dev

Note

Pull request successfully updated 🚀
Preview updated with drive-server-wip-dev:preview-274-c2c3a96e3ef400219b757266e3dd352be29f2874

@apsantiso
Copy link
Contributor Author

apsantiso commented Apr 15, 2024 •
edited by inxtci
Loading

/check-preview notify=true

Tip

Preview was successfully deployed

@internxt internxt deleted a comment from inxtci Apr 15, 2024
@internxt internxt deleted a comment from inxtci Apr 15, 2024
@apsantiso
Copy link
Contributor Author

apsantiso commented Apr 15, 2024 •
edited by inxtci
Loading

/update-preview drive-server-wip preview-274-c2c3a96e3ef400219b757266e3dd352be29f2874 imageSuffix=-dev

Note

Pull request successfully updated 🚀
Preview updated with drive-server-wip-dev:preview-274-c2c3a96e3ef400219b757266e3dd352be29f2874

@apsantiso
Copy link
Contributor Author

apsantiso commented Apr 15, 2024 •
edited by inxtci
Loading

/update-preview drive-server preview-427-c1e69124a1fe2795c613dedd0dfe28f51151609f imageSuffix=-dev

Note

Pull request successfully updated 🚀
Preview updated with drive-server-dev:preview-427-c1e69124a1fe2795c613dedd0dfe28f51151609f

@internxt internxt deleted a comment from inxtci Apr 15, 2024
@internxt internxt deleted a comment from inxtci Apr 15, 2024
@internxt internxt deleted a comment from inxtci Apr 15, 2024
@internxt internxt deleted a comment from inxtci Apr 15, 2024
@inxtci
Copy link

inxtci commented Apr 15, 2024

Caution

Pull request update failed with following error:
error: unable to find container named "payments"

@inxtci
Copy link

inxtci commented Apr 15, 2024

Caution

Pull request not updated after 5 minutes 💥
We tried to update the image with payments-dev:preview-52-a1ce7ca75a0f3919cc51b61d69978f22ebb99c03
If the arguments are correct, please check the logs. Or try again with another deployment, tag, or imageSuffix configuration.

@apsantiso apsantiso force-pushed the feat/update-user-tiers-on-webhooks branch from a1ce7ca to f5021f0 Compare April 15, 2024 13:39
@inxtci
Copy link

inxtci commented Apr 15, 2024

Caution

Pull request update failed with following error:
error: unable to find container named "payments"

@inxtci
Copy link

inxtci commented Apr 15, 2024

Caution

Pull request not updated after 5 minutes 💥
We tried to update the image with payments-dev:preview-52-f5021f01b9531363383a48651f299f522d9ce436
If the arguments are correct, please check the logs. Or try again with another deployment, tag, or imageSuffix configuration.

@apsantiso
Copy link
Contributor Author

apsantiso commented Apr 15, 2024 •
edited by inxtci
Loading

/update-preview drive-server-wip preview-274-c2c3a96e3ef400219b757266e3dd352be29f2874 imageSuffix=-dev

Note

Pull request successfully updated 🚀
Preview updated with drive-server-wip-dev:preview-274-c2c3a96e3ef400219b757266e3dd352be29f2874

@apsantiso
Copy link
Contributor Author

apsantiso commented Apr 15, 2024 •
edited by inxtci
Loading

/update-preview drive-server preview-427-c1e69124a1fe2795c613dedd0dfe28f51151609f imageSuffix=-dev

Note

Pull request successfully updated 🚀
Preview updated with drive-server-dev:preview-427-c1e69124a1fe2795c613dedd0dfe28f51151609f

@sg-gs sg-gs merged commit 8a83936 into master Apr 28, 2024
4 checks passed
@sg-gs sg-gs deleted the feat/update-user-tiers-on-webhooks branch April 28, 2024 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SimonGodefroid SimonGodefroid SimonGodefroid left review comments

@sg-gs sg-gs sg-gs approved these changes

@xabg2 xabg2 Awaiting requested review from xabg2

Assignees

@apsantiso apsantiso

Labels
deployed enhancement New feature or request preview ready-for-preview
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@apsantiso @sw-wayner @inxtci @SimonGodefroid @sg-gs