Account created on 24 April 2011, almost 13 years ago
#

Merge Requests

More

Recent comments

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

@AlfTheCat,

So you want visitor attributes as token of the current user? I think this can be added.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

I am adding unit test under this issue too. The tests should not change the code, only adds tests.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

I am still working on the revokeGrants functionality. I want to be able to revoke grants across multiple profiles. I also want to respect the existing delete memebership.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

Add tests. Group 1.x is the most popular version of Group.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

My main contribution was unit test. I also made small changes. I did remove submitRowForm, because the Authorization module does this for us, just not in parent::submitRowForm().

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

I would like to hold off on merging this until ✨ authorization_group_groups field Active is ready for review. I intend to release Authorization 1.4, Group Authorization 8.x-1.1, and 2.0 on the same day.

Group Authorization is the only module effected by the update; there are no other consumer projects.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

@Rajas Samse,

Please don't commit to my branch. If you don't use Drupal you shouldn't be submitting patches.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

bluegeek9 β†’ created an issue.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

bluegeek9 β†’ created an issue.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

Adds Project page to Related Content.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

@GeMiSKy,

I have updated the dev release with a fix. Can you verify it solved the problem for you?

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

I was able to reproduce the error. There is a canonical route in the AuthorizationProfile, but if you go to the URL a 404 is returned.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

Adds reference to custom tokens

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

I think it will be better to return the canonical URL of the entity.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

I want to give an update. I am working on ✨ Drupal Roles Consumer - store mappings in user.data Active and then plan to work on this.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

bluegeek9 β†’ made their first commit to this issue’s fork.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

@DuneBL,

I am confused by this error. MessageTemplates are not ContentEntities, so it would not have a isDefaultRevision() method. Are you able to post the view here?

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

You need to add an entry to hook_token_info for user-render.
In hook_tokens something like this:

case 'user-render':
  $entity = $message->getOwner();
  $view_builder = \Drupal::entityTypeManager()->getViewBuilder($entity->getEntityTypeId());
  $message_view = $view_builder->view($entity, 'teaser');
  $replacements[$original] = \Drupal::service('renderer')->renderRoot($message_view);
  break;
πŸ‡ΊπŸ‡ΈUnited States bluegeek9

Hi @GeMiSKy,

Thank you for reporting this issue. I hope to have it resolved soon.

Which version of Drupal are you using?

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

I think Message UI β†’ is an appropriate solution.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

Twig templates are not where want them. You want to use tokens in the message template

Author URL
[message:author:url]

Author Name
[message:author:account-name]

Author's picture
[message:author:user_picture]

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

The tests fail because $language_manager->getCurrentLanguage() returns null. I think this what happens when there is only the default language.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

bluegeek9 β†’ made their first commit to this issue’s fork.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

@liquidcms,

Did it work? Since you are writing custom code, I don't see a reason why you couldn't use the template and tokens to create the text presented to the user.

The Message entity does not have a body, so you will need to add a field to hold your custom text.

If you are looking for something less than 100% end user customized, you might be able to use token callbacks.

https://www.drupal.org/docs/extending-drupal/contributed-modules/contrib... β†’

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

Grammar

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

Clarity

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

adds a link to the user page. 

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

removes hyphen

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

Can you share the Template body? You might need to use a token callback.

https://www.drupal.org/docs/extending-drupal/contributed-modules/contrib... β†’

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

I looked at the patch. The event is dispatched after the message it send. How do you plan to use this event? I don't think I understand the value.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

bluegeek9 β†’ made their first commit to this issue’s fork.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

@liquidcms,

Take a look here:
https://www.drupal.org/docs/extending-drupal/contributed-modules/contrib... β†’

You might need to ! to all html !{my token}

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

@drupalfan2,

What do you want your custom code to do? Change who will receive the message? If you want to change how the message is rendered, you can use `hook_message_view` or `hook_message_view_alter`.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

@minorOffense,

If you are not using flags to let users subscribe, but are maintaining who should be notified separately, have you looked at Message Notify β†’ . I will look more into this error.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

bluegeek9 β†’ made their first commit to this issue’s fork.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

The message templates use tokens. Adding field formatters would be a feature request to the message project β†’ .

The Link is clipped because of how Core works. You found that you can use the title and uri from the link.

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

This is the only message I could find that is similar to the one you reported. It may have changed since the issue was reported.

Could not send notification using the "email" notifier.

This happens inside the send method:

$notifier->send($message);

Is the error still occurring?

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

Which modules do you have installed? Did you write a custom module?

πŸ‡ΊπŸ‡ΈUnited States bluegeek9

bluegeek9 β†’ made their first commit to this issue’s fork.

Production build https://api.contrib.social 0.61.6-2-g546bc20