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: Bump app version to 1.1.24, fix MaBulle icon and fix MaBulle build #1188

Merged
merged 3 commits into from
Mar 21, 2024

Conversation

Ldoppea
Copy link
Member

@Ldoppea Ldoppea commented Mar 12, 2024

### ✨ Features

*

### πŸ› Bug Fixes

* Fix MaBulle app icon's margins
* Fix MaBulle build with IAP feature disabled

### πŸ”§ Tech

*

@Ldoppea Ldoppea requested review from acezard and zatteo as code owners March 12, 2024 09:57
@Ldoppea Ldoppea changed the title Release/1.1.24 feat: Bump app version to 1.1.24, fix MaBulle icon and fix MaBulle build Mar 12, 2024
Ldoppea added a commit that referenced this pull request Mar 12, 2024
In b056a90 we added code to
dynamically remove IAP package using white_label mechanism

This correctly removed the IAP package and allowed to build the app

But since then, we added some react-native components that use the
module and would break when removing the IAP package (react-native
module is available, but the native part is not loaded)

To make them work without IAP package, we choose to move the import
behind a proxy file. Then we can override this file using white_labels
and instead import stubbed methods

The mandatory methods are `withIAPContext` and `useIAP` that wrap the
entire components and `getSubscriptions` that is used to enable/disable
IAP features

Other methods can still be exported as is as they would never be called
when IAP features are disabled

Related commit: b056a90
Related PR: #1188
Ldoppea added 3 commits March 12, 2024 11:22
In b056a90 we added code to
dynamically remove IAP package using white_label mechanism

This correctly removed the IAP package and allowed to build the app

But since then, we added some react-native components that use the
module and would break when removing the IAP package (react-native
module is available, but the native part is not loaded)

To make them work without IAP package, we choose to move the import
behind a proxy file. Then we can override this file using white_labels
and instead import stubbed methods

The mandatory methods are `withIAPContext` and `useIAP` that wrap the
entire components and `getSubscriptions` that is used to enable/disable
IAP features

Other methods can still be exported as is as they would never be called
when IAP features are disabled

Related commit: b056a90
Related PR: #1188
@Ldoppea Ldoppea merged commit 5da0a29 into master Mar 21, 2024
3 checks passed
@Ldoppea Ldoppea deleted the release/1.1.24 branch March 21, 2024 09:32
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