Skip to content

Implementing graphQL in Legacy Stage#205

Open
malaypandey-technoforte wants to merge 13 commits into
tf_nira_devfrom
tf_nira_dev_graphql
Open

Implementing graphQL in Legacy Stage#205
malaypandey-technoforte wants to merge 13 commits into
tf_nira_devfrom
tf_nira_dev_graphql

Conversation

@malaypandey-technoforte
Copy link
Copy Markdown
Contributor

No description provided.

.setLatestTransactionTypeCode(RegistrationTransactionTypeCode.LEGACY_DATA.toString());
registrationStatusDto.setRegistrationStageName(stageName);
//Setting the latest transaction status code (latest_trn_status_code) to IN_PROGRESS.
registrationStatusDto.setLatestTransactionStatusCode(RegistrationTransactionStatusCode.IN_PROGRESS.toString());
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please set substatus code and status code and status comment also in all the scenrios

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated subStatusCode, statusCode and statusComment where needed for any exceptions it is updated by updateDTOsAndLogError() method.

try {
legacyDataProcessor.connectAndSubscribe();
} catch (Exception e) {
System.out.println("Exception on connecting");
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove sysout and loggers

Copy link
Copy Markdown
Contributor Author

@malaypandey-technoforte malaypandey-technoforte Nov 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Print statement removed.

Copy link
Copy Markdown
Contributor

@SowmyaGoudar SowmyaGoudar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please do the requested changes

@malaypandey-technoforte
Copy link
Copy Markdown
Contributor Author

Implemented the requested changes.

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