Skip to content
This repository has been archived by the owner on Feb 27, 2024. It is now read-only.

contributors page #74

Merged
merged 19 commits into from
Dec 15, 2014
Merged

contributors page #74

merged 19 commits into from
Dec 15, 2014

Conversation

samsonasik
Copy link
Member

No description provided.

@samsonasik samsonasik changed the title [wip] contributor page [wip] contributors page Dec 8, 2014
@samsonasik
Copy link
Member Author

update :
1 console command to get contributors :
1-contributor
2 the page view
2-the-page

@acelaya
Copy link
Contributor

acelaya commented Dec 14, 2014

Awesome! Looks really good.

@samsonasik
Copy link
Member Author

thanks, not sure about testing consolecontroller as testing access make call to github api make limited access. ready for review ;)

@samsonasik samsonasik changed the title [wip] contributors page contributors page Dec 14, 2014
@acelaya
Copy link
Contributor

acelaya commented Dec 14, 2014

Maybe the http client or at least the adapter should be injected. We could
use the Test adapter and a hard-coded github response just to make sure our
code works.
What do you think?

Alejandro Celaya Alastrué
www.alejandrocelaya.com
El 14/12/2014 15:58, "Abdul Malik Ikhsan" notifications@github.com
escribió:

thanks, not sure about testing consolecontroller as testing access make
call to github api make limited access. ready for review ;)


Reply to this email directly or view it on GitHub
#74 (comment).

@samsonasik
Copy link
Member Author

Yes, I will try to inject the adapter. oh, yes, there is Test adapter, I will try it.

@samsonasik
Copy link
Member Author

I've added test for ConsoleController using Test Adapter. not 100% tested for now but I think it is ok for now ;)

@acelaya
Copy link
Contributor

acelaya commented Dec 15, 2014

I like how it looks now.

samsonasik added a commit that referenced this pull request Dec 15, 2014
samsonasik added a commit that referenced this pull request Dec 15, 2014
* contributor-page:
  #74 : add note to CHANGELOG.md
  add assertion
  added test for consolecontroller
  patch on dispatch for layout choose
  inject the controller with Http Client
  fixes cs
  update .phpcs
  fixes travis build
  make $contributors pass to ContributorsController
  fix forgotten 2columns layout
  fix page
  update
  fixes cs
  patch cs
  remove left bar for contribution page using "skipWelcome" layout variable
  using Zend navigation to build nav menu
  added navigation
  added test for contributors page
  fixes cs
  contributor page
samsonasik added a commit that referenced this pull request Dec 15, 2014
Forward port #74

* master:
  #74 : add note to CHANGELOG.md
  add assertion
  added test for consolecontroller
  patch on dispatch for layout choose
  inject the controller with Http Client
  fixes cs
  update .phpcs
  fixes travis build
  make $contributors pass to ContributorsController
  fix forgotten 2columns layout
  fix page
  update
  fixes cs
  patch cs
  remove left bar for contribution page using "skipWelcome" layout variable
  using Zend navigation to build nav menu
  added navigation
  added test for contributors page
  fixes cs
  contributor page
@samsonasik samsonasik merged commit 7948ab7 into sitrunlab:master Dec 15, 2014
@samsonasik samsonasik deleted the contributor-page branch December 15, 2014 08:58
@samsonasik
Copy link
Member Author

@mockiemockiz thanks for the design
@acelaya thanks for the recommendation.

It's merged to master : 3ca63bf
Forwarded to develop : c36303c

Awesome. It's online now : http://learnzf2.sitrun-tech.com/contributors ;)

@samsonasik samsonasik mentioned this pull request Dec 15, 2014
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.

3 participants