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

fix: Uses guard instead of redirection to reach onboarding page when opening the mobile app #3438

Merged
Prev Previous commit
Next Next commit
Update src/app/core/services/spender-onboarding.service.ts
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
bistaastha and coderabbitai[bot] authored Jan 23, 2025
commit fd6a096a1a06418483b4595a37f4d912b9f95d63
10 changes: 7 additions & 3 deletions src/app/core/services/spender-onboarding.service.ts
Original file line number Diff line number Diff line change
@@ -126,10 +126,14 @@ export class SpenderOnboardingService {
}

private checkCardFeedEnabled(orgSettings: OrgSettings): boolean {
const visaSettings = orgSettings.visa_enrollment_settings;
const mastercardSettings = orgSettings.mastercard_enrollment_settings;
const amexSettings = orgSettings.amex_feed_enrollment_settings;

return (
(!!orgSettings.visa_enrollment_settings.allowed && !!orgSettings.visa_enrollment_settings.enabled) ||
(!!orgSettings.mastercard_enrollment_settings.allowed && !!orgSettings.mastercard_enrollment_settings.enabled) ||
(!!orgSettings.amex_feed_enrollment_settings.allowed && !!orgSettings.amex_feed_enrollment_settings.enabled)
(visaSettings && !!visaSettings.allowed && !!visaSettings.enabled) ||
(mastercardSettings && !!mastercardSettings.allowed && !!mastercardSettings.enabled) ||
(amexSettings && !!amexSettings.allowed && !!amexSettings.enabled)
);
}

Loading