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

Change Like IdnoPlugin with link types #2431

Closed
wants to merge 13 commits into from
Closed

Change Like IdnoPlugin with link types #2431

wants to merge 13 commits into from

Conversation

johanbove
Copy link
Contributor

@johanbove johanbove commented May 18, 2019

Adding a bookmark as a "like" or a "repost" from within the edit form.

Here's what I fixed or added:

Added two check-boxes in the "Like" edit form to set the Bookmark url as a like-of and/or a repost-of url.

Annotation 2019-05-18 174803

Here's why I did it:

Personal need to more easily publish "likes" and "reposts" from within the Like plugin.
And also see Issue #1652.

Checklist:

  • This pull request addresses a single issue
  • If this code includes interface changes, I've included screenshots in this Pull Request thread
  • I've adhered to Known's style guide (these codesniffer rules might help!)
  • My git branch is named in a descriptive way - i.e., yourname-summary-of-issue
  • I've tested my code in-browser
  • My code contains descriptive comments
  • I've added tests where applicable, and...
  • I can run the unit tests successfully.

Adding a bookmark as a "like" or a "repost" from within the edit form.
mapkyca and others added 11 commits May 18, 2019 21:07
Starting with the simple stuff, lets make sure that we don't just make every variable public.
Die is often used to debug, and can be committed by accident. Lets stop that.
Die is often used to debug, and can be committed by accident. Lets stop that.
Adding a bookmark as a "like" or a "repost" from within the edit form.
Fixes resetting the likeof and repostof values
@johanbove
Copy link
Contributor Author

Resolved the merge conflict and fixed the issue with unchecking the "type" selector checkboxes.

@johanbove
Copy link
Contributor Author

This change is - probably - breaking the pubsub integration.

Fixes issue with pubsub access to Like.php
@johanbove
Copy link
Contributor Author

Resolved the issue with the pubsub POSTS failing. Taking baby-steps here...

@mapkyca
Copy link
Member

mapkyca commented Jul 21, 2019

I think the change to bookmarks was deliberate, but I'm aware it causes problems. I'll let @benwerd weigh in on this

@mapkyca
Copy link
Member

mapkyca commented Sep 4, 2019

Closing, as this functionality has been moved out of core. Resubmit there, although above conversation still applies...

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

Successfully merging this pull request may close these issues.

2 participants