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

[Bug] topicStatus needs to be changed to topic-status #4

Closed
fushnisoft opened this issue Jun 5, 2014 · 8 comments
Closed

[Bug] topicStatus needs to be changed to topic-status #4

fushnisoft opened this issue Jun 5, 2014 · 8 comments

Comments

@fushnisoft
Copy link

It looks like there was a change recently in discourse that is not compatible with this plugin.

discourse/discourse@bb0cf87

As mentioned here.

I had a quick look and I guess that topic.js.handlebars needs to be updated. I don't know if you would want to somehow maintain backwards compatibility or I would have changed it myself.

@communiteq
Copy link
Owner

We update the plugin on a regular basis to account for new Discourse versions, but we only do that when the change is made in a numbered release. So this change will be made when 0.9.9.7 is released.

@JimmydeLeeuw
Copy link

Heya, is the bug fixed with that you actually have to retype in the adress else the ads wont show up. They only show up when someone navigates to a post directly now when switching in between posts. Please fix this!

@fushnisoft
Copy link
Author

@discoursehosting I just tested you latest update, thanks! It fixes the topicStatus error but I think you missed a customHTML somewhere. I now see this:

Handlebars error: Could not find property 'customHTML' on object <Discourse.DiscoveryController:ember1490>.

@communiteq
Copy link
Owner

This is a change that has been made AFTER v0.9.9.7
Because Discourse still changes to much, we only cater for exact numbered releases.

So instead of running against master, which changes 20 times per day, you should check out v0.9.9.7 by using
git pull origin v0.9.9.7

@fushnisoft
Copy link
Author

Could you have a dev branch of the adsense plugin that is updated whenever and master remains true to the current discourse release? It is so simple to update a docker install now that it gets tempting to just try it out.

@communiteq
Copy link
Owner

We currently only update the plugin after a new numbered release, so that wouldn't be very useful. This time could be better spent in making Discourse being more forgiving to breaking changes that cause plugins to break so often.

@nlalonde
Copy link

We bumped the Discourse version again today, since a security bug was fixed.

@communiteq
Copy link
Owner

Updated the plugin. Thanks for notifying @nlalonde !

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

4 participants