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

Follow button error on profile 'most reblogged' sidebar. #1145

Closed
nolyoly opened this issue Dec 2, 2017 · 4 comments
Closed

Follow button error on profile 'most reblogged' sidebar. #1145

nolyoly opened this issue Dec 2, 2017 · 4 comments
Assignees
@jm90m
Labels
bug
Milestone
v2.2

Comments

@nolyoly
Copy link

nolyoly commented Dec 2, 2017

Today I noticed an error with the sidebar when viewing your own profile. The sidebar that says 'Most Reblogged' is displaying the 'Follow' button on users I am already following.

Below is an attached screenshot. All of these should say 'Following' instead of 'Follow'
image.png

The browser I'm using is Chromium 62 on Windows 7. I have not been able to test if this is happening in any other browsers.

I love Busy and will continue to report any bugs I find, major or not, as they all will help improve the platform!


Posted on Utopian.io - Rewarding Open Source Contributors
@espoem
Copy link
Contributor

espoem commented Dec 2, 2017

I can confirm it.

@jm90m jm90m self-assigned this Dec 2, 2017
@jm90m
Copy link
Contributor

jm90m commented Dec 2, 2017

hmm.... not too sure what happened with this but I'll take a look at it next week probably, thanks for reporting it.

@jm90m
Copy link
Contributor

jm90m commented Dec 3, 2017

Looks like this issue is happening throughout the entire app where the Follow button is used....

@jm90m jm90m added this to the v2.2 milestone Dec 4, 2017
@jm90m jm90m added the bug label Dec 4, 2017
jm90m added a commit that referenced this issue Dec 4, 2017
@jm90m
use targetUsername instead of username in userActions
d28c764 
* Fixes #1145, #1148
@jm90m jm90m mentioned this issue Dec 4, 2017
Merged
jm90m added a commit that referenced this issue Dec 4, 2017
@jm90m
use targetUsername instead of username in userActions (#1155)
be4adaf 
* Fixes #1145, #1148
Sekhmet pushed a commit to Sekhmet/busy that referenced this issue Dec 8, 2017
@jm90m @Sekhmet
use targetUsername instead of username in userActions (busyorg#1155)
596a9c9 
* Fixes busyorg#1145, busyorg#1148
@Sekhmet
Copy link
Contributor

Sekhmet commented Dec 8, 2017

Fixed in #1168.
It should work properly on busy.org.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jm90m jm90m

Labels
bug
Projects
None yet
Milestone
v2.2
Development

No branches or pull requests
4 participants
@nolyoly @Sekhmet @jm90m @espoem