Skip to content
This repository was archived by the owner on Dec 30, 2025. It is now read-only.

[Frontend - User] List the people you follow#207

Open
ardumont wants to merge 1 commit into4clojure:developfrom
ardumont:develop
Open

[Frontend - User] List the people you follow#207
ardumont wants to merge 1 commit into4clojure:developfrom
ardumont:develop

Conversation

@ardumont
Copy link

Hi,

From this evo i opened: #205

  • add some small tests around the existing "follow" functions
  • create a GET /user/:username/following resource to expose a page that lists the people :username follows.
    Basic page with readonly access that exposes the image, the name and the number of solved problems.

Bye

@ghost
Copy link

ghost commented Aug 21, 2012

is this in yet?

@Raynes
Copy link
Contributor

Raynes commented Aug 21, 2012

I don't think so. Lots of things were updated after this pull request and it can no longer be automatically merged. I'm also unclear on why he added lazytest. I don't think we want that.

Either way, @amalloy is the final word on the matter.

@ghost
Copy link

ghost commented Aug 26, 2012

thanks for the reply, I only wanted to know if there's a "List the people you follow" feature, this seemed to be it

Create the page to list in readonly mode the people someone follows.

This is the bare minimum to list as this is a startup step.
Also, this could be moved in the user profile page.
@ardumont
Copy link
Author

ardumont commented Oct 9, 2012

Hi @Raynes, i missed the comments.
I did not know what the problems were.

I see now that adding the lazytest dependency was not a good idea.
So i forced a rebased version of the develop branch without it.

I'll see to it to refresh this pull request with the current develop.

@ardumont
Copy link
Author

ardumont commented Oct 9, 2012

Hi @Raynes @amalloy,

I've rebased the branch from the current develop (tests + manual tests ok).

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.

2 participants