Closed (fixed)
Project:
Views Flag Refresh
Version:
6.x-1.0-beta3
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
3 Mar 2011 at 09:05 UTC
Updated:
5 Jul 2020 at 11:59 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
Shadlington commentedSubbing - very interested in seeing a D7 version!
Comment #2
dgastudio commentedsub!
Comment #3
cpliakas commentedHey guys. Most of my development effort is being spent on Facet API, so I won't have time to attend to this module in the near future. Would love to get some help with this and would be more than willing to pick up a co-maintainer. I am also open to starting a sandbox project and giving access to whomever so we can facilitate the update.
Thanks,
Chris
Comment #4
rv0 commentedsub
Comment #5
walker2238 commentedSubscribe. Love this module.
Comment #6
rv0 commentedeventually i needed a quick fix, so I borrowed some code from http://drupal.org/project/block_refresh and wrote some jquery to handle the flagging event. working fine.
wish i had some more time to devote to this.
Comment #7
wolfmarter commentedsubscirbing
i would also really like to see a d7 port.
@rv0 how did you get this done?
Comment #8
rv0 commented@wolfmarter - as i said, by borrowing code from the block_refresh project which has a D7 port... My solution isn't clean nor generic.
Comment #9
wolfmarter commentedmy problem is that the refresh block module removes the javascript in the url of the remove button
i am trying to fix this but with no luck so far
help appreciated
thx
wolfmarter
Comment #10
murzsubscirbing
Comment #11
webmasterkai commentedSubscribing. I might take a stab at creating a d7 port .
Comment #12
cpliakas commentedwebmasterkai Excellent! Thanks in advance.
Comment #13
cm_is commentedsubscribing
Comment #14
webadpro commentedI would really enjoy having a port for this also.
Comment #15
rodych commented+
Comment #16
Anonymous (not verified) commentedsubscribing.
I need to figure this out, so for now I'll look at using the block_refresh module, but I may just write something custom.
Comment #17
cm_is commentedany update on this one? are you working on this? regards cm
Comment #18
webmasterkai commentedYes, this is being worked on. We should have a patch in the next couple days.
Comment #19
cm_is commentedGreat to hear! really looking forward to the patch!
Comment #20
webadpro commentedThats great news!
Comment #21
Anonymous (not verified) commentedAny news on this issue?
Comment #22
thehong commentedThe team finished this port, nqbao will attach the patch soon.
Comment #23
nquocbao commentedHere is the D7 port for views_flag_refresh. Looking forward to any feedback.
Cheers
Comment #24
cpliakas commentedExcellent work! If others can RTBC this, do you want commit access?
Thanks,
Chris
Comment #25
nquocbao commentedYes, it would be great. We can create a 7.x-1.x branch then commit the patch there. Most changes are in views_flag_refresh.js
Comment #26
cpliakas commentedHi nquocbao.
I created the 6.x-1.x and 7.x-1.x branches respectively and gave you commit access. I also gave you the ability to roll releases, so do so at your will.
Thanks!
Chris
Comment #27
bjalford commentedWorks fine for me
Comment #28
nquocbao commentedHi all,
I rolled out a 7.x-1.0-beta1. http://drupal.org/node/897498/release
Cheers :)
Comment #29
webmasterkai commentedComment #30
cpliakas commentedThanks nquocbao.
Just an FYI I didn't merge your changes into the 7.x-1.x branch. Sorry for not being clear on this. Did you get a chance to merge your stuff in prior to cutting the tag?
Thanks,
Chris
Comment #31
nquocbao commentedHi cpliakas,
I merged my changes before tagging it. Here is the commit: http://drupal.org/commitlog/commit/15578/41e5e026cd621c3b171c3a1292f61a5...
Comment #32
cpliakas commentedGreat! Your commit doesn't show up in the "Maintainers for Views Flag Refresh" block nor in the "View all committers" list which is why I was confused. Thanks for pointing out the commit, and this seems like a D.O. bug that should be fixed. Your name should be there!