Thanks! Fixed
I need some time to review this MR.
Who can test it manually?
Nice work! LGTM. I need some time for manual testing
Could you explain: I understand when gallery added to node LB as block. But what is your use case for using LB for render gallery?
I don't know about "code coverage" like here https://www.drupal.org/project/knowledge/issues/3526796 📌 Add Code Coverage Active
The default CI is enough for me :)
LGTM
if no more code updates are planned, I think I can commit this
so reopening as RTBC
Ah, sorry)
Since getFlagUserFlaggings() already covers the per-flag case, maybe we could simplify getAllFlaggingByUser() to just handle “all by user” and drop the “optionally narrowed by flag” bit? That way each method has a clear, distinct role.
Could you update the MR ?
@amitsedaiz is this still relevant to you?
There is sub-module flag_bookmark. Try it
You can create Pseudo Field (or twig variable) programmatically
Looks like not relevant anymore for the issue author
Looks like not relevant anymore for the issue author
Is this still relevant to you?
Could you add full steps for reproduce?
Looks like not relevant anymore for the issue author
Looks like the patch for theming was committed https://git.drupalcode.org/project/flag/-/commit/dd511975ef69acafd8381fe...
Needs check
10 months without the answer
10 months without the answer
10 months no answer
10 months no answer
Needs to fix phpcs warning
Needs reroll to MR
Needs reroll to MR
MR from 🐛 "Unflag not allowed text" not showing up Needs review looks more complex. Closing this issue as duplicate. Also adding credit for everyone
Needs new tests here. Also existing tests failed
Needs to fix phpcs warnings
tests failed
Some tests failed
Sorry, already fixed here ✨ add a service method to get all of a user's flagged entities for a given flag Active . Credits will be added
Drupal 7 is EOL. Issue will be closed, but patches are still here
Drupal 7 is EOL. Issue will be closed, but patches are still here
Drupal 7 is EOL. Issue will be closed, but patches are still here
Drupal 7 is EOL. Issue will be closed, but patches are still here
Drupal 7 is EOL. Issue will be closed, but patches are still here
Drupal 7 is EOL. Issue will be closed, but patches are still here
Drupal 7 is EOL. Issue will be closed, but patches are still here
Drupal 7 is EOL. Issue will be closed, but patches are still here
Drupal 7 is EOL. Issue will be closed, but patches are still here