Skip to content
This repository was archived by the owner on Jan 23, 2025. It is now read-only.

Conversation

@tuxing1986
Copy link
Contributor

the components under 'components' directory are not touched


<property name="game_plan_service.jar" value="${direct_service_libdir}/game_plan_service.jar"/>

<property name="user_service_facade.jar" value="${direct_service_libdir}/user_service_facade.jar"/>
Copy link
Collaborator

Choose a reason for hiding this comment

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

@tuxing1986 it is removed, because of we built from source code?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, it's removed and the user_service_facade under the service directory is also removed.

<property name="viewData.showJustForm" value="true"/>
</bean>

<bean id="dashboardJiraIssuesReportAction"
Copy link
Collaborator

Choose a reason for hiding this comment

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

@tuxing1986 these reports are related to JIRA calls, not direct database query?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

dashboardJiraIssuesReportAction will execute "dashboard_jira_issues_report" query in the database.

Copy link
Collaborator

Choose a reason for hiding this comment

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

@tuxing1986 can you restore the related logic, db query related logic is fine, we just need to remove the JIRA API calls, as we are going to shutdown the JIRA service.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

dashboardJiraIssuesReportAction will also use ServerConfiguration.JIRA_SERVER_NAME indirectly.
Hence, it will invoke the jira calls.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

dashboardJiraIssuesReportAction is for src\web\WEB-INF\report\dashboard-jira-issues-report.jsp
That jsp page will call '${ticketId}' to get the ticket.

Hence all the action, jsp file and the related js file should all be removed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

call https:/ServerConfiguration.JIRA_SERVER_NAM/browse/ticketId

Copy link
Collaborator

Choose a reason for hiding this comment

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

@tuxing1986 this seems just linked to the external service, let me double check with PM, and response to you

Copy link
Contributor Author

Choose a reason for hiding this comment

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

okay, all the issues and bug races related to the jira urls, hence they should be removed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I can not access my gmail now, please post any issue here, thanks.

<ul>
<li><a href="http://www.jboss.org/products/jbossas/docs">JBoss Documentation</a></li>
<li><a href="http://wiki.jboss.org/">JBoss Wiki</a></li>
<li><a href="http://jira.jboss.org/">JBoss JIRA</a></li>
Copy link
Collaborator

Choose a reason for hiding this comment

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

@tuxing1986 this is not required to remove

Copy link
Contributor Author

Choose a reason for hiding this comment

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

okay

* </P>
*/
private static Overview overview;
private static Overview overview;;
Copy link
Collaborator

Choose a reason for hiding this comment

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

@tuxing1986 extra ;

Copy link
Contributor Author

Choose a reason for hiding this comment

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

okay

// set project bug race fee configuration
ProjectData directProjectData = getProjectServiceFacade().getProject(currentUser,
softwareCompetition.getProjectHeader().getTcDirectProjectId());
setFixedBugRaceFee(directProjectData.getFixedBugContestFee());
Copy link
Collaborator

Choose a reason for hiding this comment

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

@tuxing1986 is this related to JIRA ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

All the bug fix is related with JIRA.

Copy link
Collaborator

Choose a reason for hiding this comment

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

@tuxing1986 I mean this value is not related to JIRA API calls, it is just general settings for project, right? if yes, I think we can keep it, it is not harmful, and useful for auditing.

} else if (keyId == 4L) {
// svn address
getViewData().setSvnURL(data);
} else if (keyId == 5L) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

@tuxing1986 this should be restored.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's related with jira things, why it should be restored?

Copy link
Collaborator

Choose a reason for hiding this comment

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

@tuxing1986 is not it just related to set the project meta info for Bug Trace, I mean an URL?

} else if (keyId == 4L) {
// svn
getViewData().getProjectGeneralInfo().setSvn(m.getMetadataValue());
} else if (keyId == 5L) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

@tuxing1986 this should be restored

metadataOperations.add(getFormData().getBudget());
metadataOperations.add(getFormData().getDuration());
metadataOperations.add(getFormData().getSvn());
metadataOperations.add(getFormData().getJira());
Copy link
Collaborator

Choose a reason for hiding this comment

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

@tuxing1986 no need to remove this

final ProjectData project = getProjectServiceFacade().getProject(DirectUtils.getTCSubjectFromSession(),
getFormData().getProjectId());

// bug race fee
Copy link
Collaborator

Choose a reason for hiding this comment

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

@tuxing1986 does this related to JIRA API calls?

case "failureContestsInput":
sortIndex = 4;
break;
case "bugraceInput":
Copy link
Collaborator

Choose a reason for hiding this comment

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

@tuxing1986 what does this related to?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

All the bugraces are related to jiras as they are post as jira issues.

formData.svn = svnUpdate;
}

// get project jira address
Copy link
Collaborator

Choose a reason for hiding this comment

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

@tuxing1986 this is not related to JIRA call?

@skyhit skyhit changed the base branch from dev to remove_jira October 26, 2017 14:22
@skyhit skyhit merged commit 5355d7e into topcoder-archive:remove_jira Oct 26, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants