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

#167294842:fix noko project tag not showing #123

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

mulondo
Copy link

@mulondo mulondo commented Aug 16, 2019

What does this PR do?

  • This PR fixes the issue of missing project tag under Noko model.

Description of Task to be completed?

  • Currently, the ESA-API is not returning the project-name and all projects-IDs are null. This PR partly corrects that issue by filling the project fields with N/A in the Noko model at the front-end just as shown in the screenshot below.

How should this be manually tested?

  • Clone the repository
  • Run yarn in the frontend to install new dependencies.
  • Run yarn start
  • Click the details icon in the right top corner, a model will appear.
  • Select the Noko on the model.

Any background context you want to provide?

The Noko project tag field was not being populated by any values, this was due to back-end issues with the Noko-API.

What are the relevant pivotal tracker stories?

#167294842

Postman Documentation

N/A

Screenshots (If applicable)

Screen Shot 2019-08-16 at 16 15 34

- fill-up the empty spaces with N/A for missing projects.

[starts: #167294842]
@mulondo mulondo added the Peer Review Peer review label Aug 16, 2019
@@ -168,14 +168,23 @@ class AutomationDetails extends PureComponent {
return appActivities.map((content, index) => (
<div key={index}>
<div className="automation-content">
<div className="content-row name">{content[contentData]}</div>
Copy link
Contributor

Choose a reason for hiding this comment

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

Replacing this line with the renderProjectTag method causes the first column of the other tabs to be empty as this is what renders their content

@09arnold 09arnold added WIP Work in progress and removed Peer Review Peer review labels Aug 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WIP Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants