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.

Sign up for GitHub

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

Jump to bottom

/me endpoints #21

Merged
merged 17 commits into from
Dec 19, 2016
Merged

/me endpoints #21

merged 17 commits into from
Dec 19, 2016

Conversation

BrandonRomano
Copy link
Contributor

@BrandonRomano BrandonRomano commented Dec 15, 2016 •
edited
Loading

Fixes #5

@coveralls
Copy link

coveralls commented Dec 15, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-11.4%) to 88.579% when pulling f675c22 on br.me into de122be on master.

BrandonRomano
BrandonRomano commented Dec 15, 2016
View reviewed changes
controllers/me_boards_suggested_controller.go
wreckerClient *wrecker.Wrecker
}

// newMeBoardsController instantiates a new MeBoardsController
Copy link
Contributor Author

Choose a reason for hiding this comment

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

newMeBoardsSuggestedController*

@coveralls
Copy link

coveralls commented Dec 15, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-10.2%) to 89.781% when pulling 7681885 on br.me into de122be on master.

@coveralls
Copy link

coveralls commented Dec 15, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-9.1%) to 90.888% when pulling 73b68bf on br.me into de122be on master.

@coveralls
Copy link

coveralls commented Dec 15, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-8.4%) to 91.574% when pulling efaa05a on br.me into de122be on master.

@coveralls
Copy link

coveralls commented Dec 16, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-7.4%) to 92.569% when pulling 916e959 on br.me into de122be on master.

@coveralls
Copy link

coveralls commented Dec 16, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-7.4%) to 92.569% when pulling ba7a0d9 on br.me into 42e89cd on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-6.5%) to 93.483% when pulling dfe9169 on br.me into 42e89cd on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-6.5%) to 93.483% when pulling dfe9169 on br.me into 42e89cd on master.

@coveralls
Copy link

coveralls commented Dec 16, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-4.9%) to 95.057% when pulling 5333a5b on br.me into 42e89cd on master.

@coveralls
Copy link

coveralls commented Dec 16, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-5.5%) to 94.505% when pulling 75d38df on br.me into 42e89cd on master.

@coveralls
Copy link

coveralls commented Dec 16, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-5.8%) to 94.176% when pulling e8d5734 on br.me into 42e89cd on master.

@coveralls
Copy link

coveralls commented Dec 19, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-3.5%) to 96.506% when pulling 417138a on br.me into 42e89cd on master.

@coveralls
Copy link

coveralls commented Dec 19, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-3.5%) to 96.506% when pulling d6a52c1 on br.me into 42e89cd on master.

@coveralls
Copy link

coveralls commented Dec 19, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-2.9%) to 97.101% when pulling d563d2a on br.me into 42e89cd on master.

@coveralls
Copy link

coveralls commented Dec 19, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-2.3%) to 97.66% when pulling 9a81aba on br.me into 42e89cd on master.

@coveralls
Copy link

coveralls commented Dec 19, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-1.8%) to 98.195% when pulling 4a0f915 on br.me into 42e89cd on master.

@coveralls
Copy link

coveralls commented Dec 19, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-1.3%) to 98.694% when pulling 5127b68 on br.me into 42e89cd on master.

@coveralls
Copy link

coveralls commented Dec 19, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-1.02%) to 98.984% when pulling c35f8bc on br.me into 42e89cd on master.

@coveralls
Copy link

coveralls commented Dec 19, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling 1a934a6 on br.me into 42e89cd on master.

BrandonRomano
BrandonRomano commented Dec 19, 2016
View reviewed changes
controllers/me_boards_controller.go

// Fetch loads the authorized users boards
// Endpoint: [GET] /v1/me/boards/
func (mbc *MeBoardsController) Fetch() (*[]models.Board, error){
Copy link
Contributor Author

Choose a reason for hiding this comment

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

is this paginated?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Doesn't appear so...

BrandonRomano
BrandonRomano commented Dec 19, 2016
View reviewed changes
controllers/me_following_interests_controller.go
}

// Fetch loads the authorized users interests
func (mfic *MeFollowingInterestsController) Fetch(optionals *MeFollowingInterestsFetchOptionals) (*[]models.Interest, *models.Page, error) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Add Endpoint: to docs

BrandonRomano
BrandonRomano commented Dec 19, 2016
View reviewed changes
controllers/me_likes_controller.go
}

// Fetch loads all of the pins that the authorized user has liked
func (mlc *MeLikesController) Fetch(optionals *MeLikesFetchOptionals) (*[]models.Pin, *models.Page, error) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Add Endpoint: to docs

BrandonRomano
BrandonRomano commented Dec 19, 2016
View reviewed changes
controllers/me_pins_controller.go
}

// Fetch loads all of the logged in user's Pins
func (mpc *MePinsController) Fetch(optionals *MePinsFetchOptionals) (*[]models.Pin, *models.Page, error) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Add Endpoint:

BrandonRomano
BrandonRomano commented Dec 19, 2016
View reviewed changes
controllers/me_search_boards_controller.go
}

// Fetch searches the logged in user's Boards
func (msbc *MeSearchBoardsController) Fetch(query string, optionals *MeSearchBoardsFetchOptionals) (*[]models.Board, *models.Page, error) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Add Endpoint:

BrandonRomano
BrandonRomano commented Dec 19, 2016
View reviewed changes
controllers/me_search_pins_controller.go
}

// Fetch searches the logged in user's Pins
func (mspc *MeSearchPinsController) Fetch(query string, optionals *MeSearchPinsFetchOptionals) (*[]models.Pin, *models.Page, error) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Endpoint:

@BrandonRomano BrandonRomano changed the title [WIP] /me endpoints /me endpoints Dec 19, 2016
@coveralls
Copy link

coveralls commented Dec 19, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling 6945534 on br.me into 42e89cd on master.

@BrandonRomano BrandonRomano merged commit 2349170 into master Dec 19, 2016
@BrandonRomano BrandonRomano deleted the br.me branch December 19, 2016 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@BrandonRomano @coveralls