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

Added the ability to comment on a map #42

Closed
wants to merge 1 commit into from

Conversation

YosoraLife
Copy link

It only a small change, but i think more people could benefit from this.

I am using Waymark to display motorcycle routes on my websites. And i thought it would be nice if people could comment/discuss those routes. So since Waymaps is a custom post type, I added the default wordpress ability to comment on a map.

Added comments support
@morehawes
Copy link
Collaborator

Hi @YosoraLife,

Thank you for submitting this, I actually think allowing comments for Maps makes complete sense and would make a nice addition to the plugin :)

My only concern with this change is that by enabling this capability by default, some users with existing Maps might not realise and may unknowingly start publishing Maps with comments enabled by default (depending on the Discussion Settings > Default post settings > Allow people to submit comments on new posts WordPress Setting).

Adding a Waymark Setting to enable/disable comments would be one way around this.

Do you have any thoughts?

Cheers,

Joe

@morehawes morehawes self-assigned this May 17, 2024
@morehawes morehawes added enhancement New feature or request question Further information is requested labels May 17, 2024
@YosoraLife
Copy link
Author

YosoraLife commented May 18, 2024

Hi Joe,

I understand where you're coming from. And I agree that code changes should not affect already published maps. But this would only affect new maps. And then again, for newly published maps, this would only affect those who have enabled commenting on their site.

I have a bit of mixed feelings about an extra setting. On the one hand, you have already built in many settings (which I have also gratefully made use of) and there will undoubtedly be people who want comments on their posts but not comments on the maps (although they can also disable comments per map).

On the other hand, it is an extra setting that ultimately needs to be maintained. And it's just a small adjustment, but every little bit adds up. It would be a shame if it all became so much that the plugin could no longer be maintained.

Cheers,
Remco

@morehawes
Copy link
Collaborator

Hi Remco,

Thank you for your thoughts :) I have given this some more thought and I am definitely leaning towards enabling this by default.

While I don't love the idea of creating a scenario where the user might unknowingly create a comments-enabled Map, I think the benefits of tapping into the WordPress post commenting system makes a lot of sense and might outweigh those drawbacks.

Cheers,

Joe

@YosoraLife
Copy link
Author

YosoraLife commented May 21, 2024

Hi Joe, of course, your welcome :)

Would it also be an idea to add 'excerpt' to the supports array? Because now in a search query I get this as a result:

Screenshot 2024-05-21 211800

While its text from the actual page it's not really in context. And it would be nice to have some text that actually makes some sense when you read it (and to have some control on what's displayed as excerpt). And unlike with the comment this doesn't affect anyone if there not using it, it just another possibility than can be used if you want to.

@morehawes
Copy link
Collaborator

Hi @YosoraLife,

Would it also be an idea to add 'excerpt' to the supports array?

I am not apposed to this, but I am a little weary of overwhelming the Map Create/Edit interface by enabling too many of the Post capabilities.

I think instead of a PR this might be better broken up into a separate Issue for further discussion.

I am traveling at the moment and will give these some more time when I return.

Thanks again!

Joe

@YosoraLife
Copy link
Author

I think instead of a PR this might be better broken up into a separate Issue for further discussion.

No problem, i'll open a separate issue. Enjoy your travels ;)

@YosoraLife YosoraLife mentioned this pull request May 23, 2024
morehawes added a commit that referenced this pull request Jun 10, 2024
Enable or disable various WordPress features of the Map Custom Post Type, including the new ability to enable comments and excerpt integration. This Setting is Available in Settings > Advanced > Post Type. Thanks to [YosoraLife](https://github.com/YosoraLife) for requesting this ([here](#42) and [here](#44)).
@morehawes
Copy link
Collaborator

Hi @YosoraLife,

I have released version 1.2.0 of the plugin which includes the ability to enable comment support.

Please do let me know if you encounter any issues!

Thanks again for the suggestion.

Joe

@morehawes morehawes closed this Jun 10, 2024
@morehawes morehawes removed the question Further information is requested label Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

Successfully merging this pull request may close these issues.

None yet

2 participants