Skip to content
This repository has been archived by the owner on Aug 5, 2021. It is now read-only.

Updated help-wanted-card.template.html to show agency_id next to agency logo #570

Merged
merged 1 commit into from
Jun 6, 2018

Conversation

ctrimm
Copy link
Contributor

@ctrimm ctrimm commented Jun 6, 2018

Summary

This PR fixes/implements the following bugs/features

This will make it a little bit easier for end users to be able to tell what Agency the Help Wanted task belongs to.

Test plan

I cloned the repo, made sure I was on the proper node version (6.11.0 at time of PR according to .nvmrc), installed the node_modules, and ran the development server. After that, I updated a couple lines of code with conditional angular logic to show/hide the <span class="help-wanted-agency-name"> tag if the agency_id is set on the item object that is passed into the help-wanted-card.template.html file.

I also -

  • Manually tested the code on my local machine
  • Ran unit tests -
Finished in 2.681 secs / 2.918 secs @ 21:38:28 GMT-0600 (MDT)

SUMMARY:
✔ 119 tests completed

=============================== Coverage summary ===============================
Statements   : 73.99% ( 711/961 )
Branches     : 29.57% ( 55/186 )
Functions    : 56.38% ( 137/243 )
Lines        : 72.77% ( 628/863 )
================================================================================
  • Ran the end-to-end tests - 1 spec, 0 failures - Finished in 3.364 seconds

Below are two examples of what the UI looks like.
screen shot 2018-06-05 at 6 56 20 pm
screen shot 2018-06-05 at 6 56 26 pm

Code formatting

Closing issues

Fixes #493

@ctrimm
Copy link
Contributor Author

ctrimm commented Jun 6, 2018

Can we re-run this circleCI test @DanielJDufour - I do not have permissions to do so. I think this is a transient failure. I re-ran the tests locally on this branch and they all passed again...

@DanielJDufour
Copy link
Contributor

Awesome! @ctrimm , I can confirm that this is an issue with the tests, not your code. Thanks so much for your help!

(I'll work on updating the tests, so we don't run into this problem again)

Copy link
Contributor

@DanielJDufour DanielJDufour left a comment

Choose a reason for hiding this comment

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

Great job!

@DanielJDufour DanielJDufour merged commit 9f73749 into GSA:master Jun 6, 2018
@ctrimm
Copy link
Contributor Author

ctrimm commented Jun 7, 2018

@DanielJDufour - I see that the code is live, thanks.

However, the 'Help Wanted page agency name missing next to agency logo' item still is being returned in the help-wanted.json. Which is from from the GET request at this URL https://code.gov/assets/help-wanted.json.

@DanielJDufour
Copy link
Contributor

@ctrimm, good catch! I'll fix that today!

@DanielJDufour
Copy link
Contributor

@ctrimm , I just updated the Help Wanted page. Thanks again!

@ctrimm
Copy link
Contributor Author

ctrimm commented Jun 7, 2018

No problem @DanielJDufour. Thank you for the quick communication. I was in shock with how quickly this was addressed - definitely makes me want to pick up more tasks to contribute to.

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.

Help Wanted page agency name missing next to agency logo
2 participants