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

Partners column missing in project list widget #15

Closed
zzgvh opened this issue May 23, 2012 · 6 comments
Closed

Partners column missing in project list widget #15

zzgvh opened this issue May 23, 2012 · 6 comments

Comments

@zzgvh
Copy link
Contributor

zzgvh commented May 23, 2012

The Partners column in the project list widget shows no organisations

Example: http://www.akvo.org/rsr/widget/project-list/organisation/42/

Edit: @adriancollier 29th October 2012.

There are additional requirements that are not mentioned here that while they do not relate specifically to fix, are relevant to the code being written. These include standard items such as Widgets should be viewable ion a variety of browsers and systems. If further information is needed on these standard requirements, please contact @adriancollier.

  • The widget should contain the following columns: Project, Location, Status/Last Updated, Partners, Total Budget/Funding Needed
  • All columns visible on the Widget should be populated by relevant content from Akvo RSR
  • The Partners Column should contain a list of all partner organisations for this project
  • If a partner is linked to the project under 2 or more roles, then only a single entry for this partner should be displayed on the Partners list in the widget column

This work has been estimated at 1 day of development time. Formal initiation of this work will be given shortly along with the assigning of this issue.

A comment at this time will be added to provide the estimated delivery time.

@zzgvh
Copy link
Contributor Author

zzgvh commented May 23, 2012

Also filed in PT:

https://www.pivotaltracker.com/story/show/30067489

@ghost ghost assigned kardan Apr 9, 2013
kardan added a commit that referenced this issue Apr 12, 2013
Project partners are now listed with working
links.

. #15
kardan added a commit that referenced this issue Apr 12, 2013
kardan added a commit that referenced this issue Apr 15, 2013
Project partners are now listed with working
links.

. #15
kardan added a commit that referenced this issue Apr 15, 2013
@kardan
Copy link
Contributor

kardan commented Apr 15, 2013

@adriancollier
Copy link
Contributor

works for me, @peeb can you do a code check please?

@ghost ghost assigned peeb Jul 2, 2013
@peeb
Copy link
Contributor

peeb commented Jul 8, 2013

Code reviewed.

Not a huge fan of trailing "" in longer lines of code. Can we perhaps use parentheses to wrap longer lines when possible?

@zzgvh
Copy link
Contributor Author

zzgvh commented Jul 9, 2013

Yah, I concur, "" is evil! And I'm not a disciple of the Church of Eighty Chars either. 😛 Given the screens we use today I'd say around 120 chars makes more sense.

@kardan
Copy link
Contributor

kardan commented Jul 9, 2013

Agree on the () .v.s \ but I'm in the Church of Eighty Chars :-)

@MichaelAkvo MichaelAkvo moved this to Done in RSR Dec 8, 2022
@MichaelAkvo MichaelAkvo added this to RSR Dec 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

4 participants