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

bugfix #1345

Closed
wants to merge 3 commits into from
Closed

bugfix #1345

wants to merge 3 commits into from

Conversation

ayushchatur
Copy link
Contributor

Implementation details:

  1. As referred from archived active jobs plugin the semantic of the changes were in bad shape. There were incomplete ``` HTML blocks in _job_details_node_view.
    Hence this was rectified along with:
  2. An additional table that was used but not required to render the ganglia graphs in _ganglia_graphs_table

Copy link
Contributor

@johrstrom johrstrom left a comment

Choose a reason for hiding this comment

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

Your commits don't seem to be associated with your GH user. Is it the right user.email git config?

@ayushchatur
Copy link
Contributor Author

@johrstrom rebased with correct author

@johrstrom
Copy link
Contributor

You still have the unknown user in the commit history.

@ayushchatur
Copy link
Contributor Author

raised another #1360

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants