I've a question. There seems to be a change to the hook
static:app.load. I've not updated the plugin tho because I have a bug when I change this in
Whenever I try to acces a page on the forum I have a :
Cannot GET /[route]
Going back to action app.load resolve this. Any clues of what is happening ? Or do I need to wait for next version to update my plugin ?
One more thing I've seen that I've forgotten the search results so there will be an update to expect and if I could include this aswell it could be nice.
static:app.loadhas a fourth parameter (callback), ex:
Version 1.2.0 is out.
- Bug fixes.
- Massive code revamp on main.js thanks to @Xevious. Would be easier to keep up-to-date. There's less request and a lot of improvement.
- Updated the ACP view.
- Compatible with next version of NodeBB, the topics will be colored on homepage for this version thanks to
While the plugin is fully functional on 0.5.0...
Major issue DO NOT USE THIS PLUGIN WITH LATEST MASTER BUILD !
Your topics will return 404 pages. Deactivating the plugin give access back.
On the code side and why there will likely be an update in not a long time but I need to be sure before.
NodeBB seems to have changed how the slugs are interpreted, so a topic called
I'm a testis accessible via
If you write
%(violet)[I'm a test]the address will become
The thing is that having the color in the url was something I considered ugly and so I used a little regex to clean the address like to be again
But what worked before is no longer working it will return a 404 because NodeBB is looking for this address to display the topic information
domain/topic/1/violet-i-m-a-testso I guess it could just be a URL rewriting thing. I also don't know why the slugs would have change to be the exact same as the topic before doing
domain/topic/1/whateverwas displaying the topic.
I did that rewriting for SEO issues and also to avoid things like
The wrong slugs are returning 404 now because of this issue.
Previously if you would include an image in your post with
!(test)it would load the entire topic again because it would make a request to
domain.com/topic/1/testand since the slug didn't matter it would reload the entire topic.
If we can figure out a better way to handle this instead of just sending 404 for wrong slugs then we can go back to ignoring slug.
@baris Isn't the problem to figure out why
So if understand I'll have to wait and see if I remove this feature from the plugin or keep it.
Maybe the solution is to be able to set custom slugs via plugin on action:topic.save
It's just the way it works, it treats it as a local image and tries to load from /topic/1/test which matches the topic route and loads the topic if we don't check the slug.
I haven't found another way to prevent it other than checking the slug and making sure its correct.