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 into
4clojure:developfrom
ardumont:develop
Open

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

Conversation

@ardumont

Copy link
Copy Markdown

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

ghost commented Aug 21, 2012

Copy link
Copy Markdown

is this in yet?

@Raynes

Raynes commented Aug 21, 2012

Copy link
Copy Markdown
Contributor

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

ghost commented Aug 26, 2012

Copy link
Copy Markdown

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

ardumont commented Oct 9, 2012

Copy link
Copy Markdown
Author

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

ardumont commented Oct 9, 2012

Copy link
Copy Markdown
Author

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