Skip to content
GitLab
Explore
Projects
Groups
Topics
Snippets
Projects
Groups
Topics
Snippets
/
Help
Help
Support
Community forum
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Register
Sign in
Toggle navigation
Menu
UPSMF
uphrh-SunbirdEd-portal-old
Commits
6c85c2f1
Commit
6c85c2f1
authored
5 years ago
by
Devesh
Browse files
Options
Download
Patches
Plain Diff
Issue #SB-13773 fix:error message changes for auto and manual merge
parent
566466ee
master
Dark_theme
SB-25589
SB-28090
aws_fileRead
contributions
dependabot/npm_and_yarn/src/app/client/decode-uri-component-0.2.2
dependabot/npm_and_yarn/src/app/decode-uri-component-0.2.2
dependabot/npm_and_yarn/src/app/express-4.17.3
dependabot/npm_and_yarn/src/app/jsonwebtoken-9.0.0
dependabot/npm_and_yarn/src/desktop/OpenRAP/decode-uri-component-0.2.2
dependabot/npm_and_yarn/src/desktop/OpenRAP/express-4.17.3
dependabot/npm_and_yarn/src/desktop/OpenRAP/jsonwebtoken-9.0.0
dependabot/npm_and_yarn/src/desktop/OpenRAP/qs-6.5.3
dependabot/npm_and_yarn/src/desktop/decode-uri-component-0.2.2
dependabot/npm_and_yarn/src/desktop/express-4.17.3
dependabot/npm_and_yarn/src/desktop/jsonwebtoken-9.0.0
dependabot/npm_and_yarn/src/desktop/qs-6.10.3
desktop-common-consumption
filters_5.1.1
gcp
gcp_test
keshavprasadms-patch-1
keshavprasadms-patch-2
release-2.10.0
release-2.10.1
release-2.10.2
release-2.10.3
release-2.4.0
release-2.4.1
release-2.4.2
release-2.5.0
release-2.5.1
release-2.6.0
release-2.6.0-loadtest
release-2.6.5
release-2.6.6
release-2.7.0
release-2.8.0
release-2.8.1
release-2.8.10
release-2.8.11
release-2.8.12
release-2.8.2
release-2.8.3
release-2.8.4
release-2.8.5
release-2.8.6
release-2.8.7
release-2.8.8
release-2.8.9
release-2.9.0
release-3.0
release-3.0-merge
release-3.0.0
release-3.0.1
release-3.0.2
release-3.0.3
release-3.0.4
release-3.0.4-sso
release-3.1.0
release-3.1.1
release-3.1.2
release-3.2.0
release-3.2.1
release-3.2.10
release-3.2.11
release-3.2.12
release-3.2.13
release-3.2.14
release-3.2.2
release-3.2.3
release-3.2.4
release-3.3.0
release-3.3.0-telemetry-fix
release-3.3.1
release-3.3.2
release-3.4.0
release-3.4.1
release-3.4.2
release-3.4.3
release-3.4.4
release-3.4.5
release-3.4.6
release-3.4.7
release-3.5.0
release-3.5.1
release-3.5.2
release-3.5.3
release-3.6.0
release-3.6.1
release-3.6.5
release-3.6.6
release-3.7.0
release-3.7.1
release-3.7.2
release-3.8.0
release-3.8.1
release-3.8.2
release-3.8.3
release-3.9.0
release-3.9.1
release-3.9.2
release-3.9.3
release-4.0.0
release-4.0.1
release-4.0.2
release-4.1
release-4.1.0
release-4.1.1
release-4.10.0
release-4.10.0.1
release-4.10.1
release-4.10.2
release-4.10.2.1
release-4.10.2.2
release-4.10.3
release-4.2.0
release-4.2.1
release-4.3.0
release-4.3.1
release-4.4.0
release-4.4.1
release-4.5.0
release-4.5.1
release-4.5.2
release-4.6.0
release-4.7.0
release-4.7.1
release-4.8.0
release-4.8.5
release-4.9.0
release-4.9.1
release-5.0.0
release-5.0.0.1
release-5.0.0.2
release-5.0.0.3
release-5.0.1
release-5.0.2
release-5.1.0
release-5.1.1
release-sonarcloud
revert-3718-copypi
revert-3842-release-2.8.7
revert-3890-release-2.8.9
revert-4427-player-cache-issue
revert-4537-SB-19763
revert-5244-sh-809
revert-5260-SB-table-3.3.0
revert-8284-release-5.0.0
sharathkashyap-patch-1
Tags unavailable
No related merge requests found
Changes
4
Hide whitespace changes
Inline
Side-by-side
Showing
4 changed files
src/app/client/src/app/modules/merge-account/components/merge-account-status/merge-account-status.component.html
+10
-4
.../merge-account-status/merge-account-status.component.html
src/app/client/src/app/modules/merge-account/components/merge-account-status/merge-account-status.component.ts
+2
-0
...ts/merge-account-status/merge-account-status.component.ts
src/app/resourcebundles/data/consumption/en.properties
+2
-1
src/app/resourcebundles/data/consumption/en.properties
src/app/routes/accountMergeRoute.js
+2
-2
src/app/routes/accountMergeRoute.js
with
16 additions
and
7 deletions
+16
-7
src/app/client/src/app/modules/merge-account/components/merge-account-status/merge-account-status.component.html
+
10
−
4
View file @
6c85c2f1
...
...
@@ -7,10 +7,16 @@
<div
class=
"sb-merged-account-content text-center"
>
<h3
class=
"sb-merged-account-body-header my-24 font-weight-bold f-18"
>
{{resourceService.frmelmnts?.lbl?.mergeAccount}}
</h3>
<p
*ngIf=
"isMergeSuccess"
class=
"sb-merged-account-body-para sb-color-success font-weight-bold py-8"
>
{{resourceService.messages?.imsg?.m0065}}. {{resourceService.messages?.imsg?.m0066}}
</p>
<p
*ngIf=
"!isMergeSuccess"
class=
"sb-merged-account-body-para sb-color-error font-weight-bold py-8"
>
{{resourceService.messages?.imsg?.m0067}}
</p>
<p
*ngIf=
"isMergeSuccess"
class=
"sb-merged-account-body-para sb-color-success font-weight-bold py-8"
>
{{resourceService.messages?.imsg?.m0065}}. {{resourceService.messages?.imsg?.m0066}}
</p>
<div
*ngIf=
"mergeType === 'manual'"
>
<p
*ngIf=
"!isMergeSuccess"
class=
"sb-merged-account-body-para sb-color-error font-weight-bold py-8"
>
{{resourceService.messages?.imsg?.m0067}}
</p>
</div>
<div
*ngIf=
"mergeType === 'auto'"
>
<p
*ngIf=
"!isMergeSuccess"
class=
"sb-merged-account-body-para sb-color-error font-weight-bold py-8"
>
{{resourceService.messages?.imsg?.m0080}}
</p>
</div>
</div>
<div
class=
"d-flex flex-jc-center mt-40"
>
<button
class=
"sb-btn sb-btn-secondary sb-btn-md w-100"
...
...
This diff is collapsed.
Click to expand it.
src/app/client/src/app/modules/merge-account/components/merge-account-status/merge-account-status.component.ts
+
2
−
0
View file @
6c85c2f1
...
...
@@ -11,6 +11,7 @@ export class MergeAccountStatusComponent implements OnInit {
@
ViewChild
(
'
modal
'
)
modal
;
isMergeSuccess
:
any
=
{};
redirectUri
:
string
;
mergeType
:
string
;
constructor
(
public
activatedRoute
:
ActivatedRoute
,
public
resourceService
:
ResourceService
)
{
}
...
...
@@ -19,6 +20,7 @@ export class MergeAccountStatusComponent implements OnInit {
this
.
activatedRoute
.
queryParams
.
subscribe
((
queryParams
)
=>
{
const
queryParam
=
{...
queryParams
};
this
.
isMergeSuccess
=
queryParam
.
status
===
'
success
'
;
this
.
mergeType
=
queryParam
.
merge_type
;
this
.
redirectUri
=
queryParam
.
redirect_uri
||
'
/resources
'
;
});
}
...
...
This diff is collapsed.
Click to expand it.
src/app/resourcebundles/data/consumption/en.properties
+
2
−
1
View file @
6c85c2f1
...
...
@@ -682,8 +682,9 @@ messages.imsg.m0063 = combine usage details of the both accounts, and
messages.imsg.m0064
=
delete the other account
messages.imsg.m0065
=
Account merge has been successfully initiated
messages.imsg.m0066
=
You will be notified once it is complete
messages.imsg.m0067
=
Account merge has not been initiated. Please try
again later
messages.imsg.m0067
=
Account merge has not been initiated. Please
re
try
from the profile menu.
messages.imsg.m0070
=
Account already exists On {instance}, If this is your account, click Merge to
messages.imsg.m0080
=
Account merge has not been initiated. Please retry by logging in to your state system
messages.imsg.m0071
=
Cancel to create new {instance} Account
messages.imsg.m0072
=
Could not merge accounts because the password entered is incorrect.
messages.imsg.m0073
=
Click OK to create a new {instance} account.
...
...
This diff is collapsed.
Click to expand it.
src/app/routes/accountMergeRoute.js
+
2
−
2
View file @
6c85c2f1
...
...
@@ -64,14 +64,14 @@ module.exports = (app) => {
u2Token
).
catch
(
err
=>
{
console
.
log
(
'
error in initiateAccountMerge
'
,
JSON
.
stringify
(
err
));
console
.
log
(
'
error detals
'
,
err
.
statusCode
,
err
.
message
);
const
query
=
'
?status=error&redirect_uri=
'
+
req
.
session
.
mergeAccountInfo
.
initiatorAccountDetails
.
redirectUri
;
const
query
=
'
?status=error&
merge_type=manual&
redirect_uri=
'
+
req
.
session
.
mergeAccountInfo
.
initiatorAccountDetails
.
redirectUri
;
req
.
session
.
mergeAccountInfo
=
null
;
const
redirectUri
=
`https://
${
req
.
get
(
'
host
'
)}
/accountMerge`
+
query
;
res
.
redirect
(
url
+
redirectUri
);
});
if
(
_
.
get
(
mergeResponse
,
'
result.result.status
'
)
===
'
SUCCESS
'
&&
mergeResponse
.
responseCode
===
'
OK
'
)
{
console
.
log
(
'
mergeResponse coming from backend
'
,
JSON
.
stringify
(
mergeResponse
));
const
query
=
'
?status=success&redirect_uri=
'
+
req
.
session
.
mergeAccountInfo
.
initiatorAccountDetails
.
redirectUri
;
const
query
=
'
?status=success&
merge_type=manual&
redirect_uri=
'
+
req
.
session
.
mergeAccountInfo
.
initiatorAccountDetails
.
redirectUri
;
console
.
log
(
'
after final success
'
,
query
);
const
redirectUri
=
`https://
${
req
.
get
(
'
host
'
)}
/accountMerge`
+
query
;
req
.
session
.
mergeAccountInfo
=
null
;
...
...
This diff is collapsed.
Click to expand it.
Write
Preview
Supports
Markdown
0%
Try again
or
attach a new file
.
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Save comment
Cancel
Please
register
or
sign in
to comment
Menu
Explore
Projects
Groups
Topics
Snippets