Infrastructure/logto
0
Fork 0
mirror of https://github.com/logto-io/logto.git synced 2025-01-06 20:40:08 -05:00
Code Issues Activity

fix(experience): hide scope list if no user scopes and resource scopes (#5840)

Browse source
This commit is contained in:
Xiao Yijun 2024-05-10 11:00:26 +08:00 committed by GitHub
parent feae37b9b1
commit 266af8c3d7
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 3 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
packages/experience/src/pages/Consent/ScopesListCard/index.tsx
View file

@ -45,15 +45,12 @@ const ScopesListCard = ({
[t, userScopes] [t, userScopes]
); );
const showTerms = Boolean(termsUrl ?? privacyUrl); // Todo @xiaoyijun remove dev feature flag and authorization agreement from this component
const showTerms = !isDevFeaturesEnabled && Boolean(termsUrl ?? privacyUrl);
// If there is no user scopes and resource scopes, we don't need to show the scopes list. // If there is no user scopes and resource scopes, we don't need to show the scopes list.
// This is a fallback for the corner case that all the scopes are already granted. // This is a fallback for the corner case that all the scopes are already granted.
if ( if (!userScopesData?.length && !resourceScopes?.length) {
!isDevFeaturesEnabled && // Todo @xiaoyijun remove dev feature flag
!userScopesData?.length &&
!resourceScopes?.length
) {
return showTerms ? ( return showTerms ? (
<div className={className}> <div className={className}>
<Trans <Trans