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.

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

TypeError: subject.badge().icon is not a function #32

Open
edgeinfinity1 opened this issue Jul 27, 2023 · 9 comments
Open

TypeError: subject.badge().icon is not a function #32

edgeinfinity1 opened this issue Jul 27, 2023 · 9 comments

Comments

@edgeinfinity1
Copy link

This happens when I try to open the notifications sector after I gained some badges given by AutoModerator.
The rest of the notifications still loaded, but nothing about newly get badges is shown.

@Lisra-git
Copy link

#35 Should fix the issue

@edgeinfinity1
Copy link
Author

#35 Should fix the issue

Thanks for that, I migrated mine and its now working fine.

@edgeinfinity1
Copy link
Author

edgeinfinity1 commented Mar 26, 2024

#35 Should fix the issue

Sorry to bother, but later this day after I did the same changes as the PR, I began recieving errors when using the search function:
image
Yes I had my flarum core and its search functions modified, but this problem never happened before. Is that my problem, or is this PR not completed yet?
Also yes, I ran npm run dev and things already.

@edgeinfinity1 edgeinfinity1 reopened this Mar 26, 2024
@Lisra-git
Copy link

Will have a look asap

@Lisra-git
Copy link

Search function seem to work on my testing instance, is there any specific change you made on your search?

@edgeinfinity1
Copy link
Author

Search function seem to work on my testing instance, is there any specific change you made on your search?

image I had the View Counts plugin in my forum, so I modified this way to show the most viewed posts on top. Could that be the cause of the problem?

@edgeinfinity1
Copy link
Author

Would it be convenient for you to merge and release this PR? Though I just looked through this plugin roughly so my understanding may be poor, I didnt see anything in PHPs defining badge a property of $userBadge. Is it possible that you made other changes on your local copy but forgot to include in this PR? I'm sorry in advance for assuming.

@Lisra-git
Copy link

Actually I don't really see how it could break your edits, the only impact is on received notification

@edgeinfinity1
Copy link
Author

Actually I don't really see how it could break your edits, the only impact is on received notification

Could you spare some time to make a release of this patch? I want to do some further tests on where the problem could be.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants