๐Ÿ‡ฎ๐Ÿ‡ณIndia @kunal_singh

Account created on 10 April 2018, about 6 years ago
#

Merge Requests

Recent comments

๐Ÿ‡ฎ๐Ÿ‡ณIndia kunal_singh

Thanks @Arijit Acharya

The MR is merged!

๐Ÿ‡ฎ๐Ÿ‡ณIndia kunal_singh

Thanks for the MR, @Indranil Roy I still see few of the things which should be added to make it work properly.

  1. We can add a schema file.
  2. We will have to add an update hook to update the configuration for existing mailbox users.
  3. We will have to add blacklist_emails in the config file as well.
๐Ÿ‡ฎ๐Ÿ‡ณIndia kunal_singh

It looks like this patch is adding a regression issue. The change here https://git.drupalcode.org/project/mobile_number/-/commit/e8939cb64cc074... is removing the hidden token value. Which results in a verification error.

Instead of looping through the elements, it should get triggered on the `click` event ( As it used to previously )

๐Ÿ‡ฎ๐Ÿ‡ณIndia kunal_singh

You can check this module:  https://www.drupal.org/project/login_time_restriction โ†’

This will not meet your exact requirements but you can alter or request for feature if needed.

๐Ÿ‡ฎ๐Ÿ‡ณIndia kunal_singh

@ben.bastow The issue is addressed here: https://www.drupal.org/project/openai/issues/3413412 ๐Ÿ› Validate function is getting called for all the plugins Fixed

๐Ÿ‡ฎ๐Ÿ‡ณIndia kunal_singh

I have initiated this module https://www.drupal.org/project/recommend_ai โ†’ for the same. Although it's in very early stage but the goal is to have a full fledged content recommendation system with personalization

๐Ÿ‡ฎ๐Ÿ‡ณIndia kunal_singh

Just realized that we have the same validateConfigurationForm function in the `Milvus` plugin and that is also restricting the form save.

๐Ÿ‡ฎ๐Ÿ‡ณIndia kunal_singh

Make sense.

Let me do the changes.

๐Ÿ‡ฎ๐Ÿ‡ณIndia kunal_singh

Mybad, Just realised that the issue is fixed in `3411462-cant-save-embeddings` branch.

@kamkejj Can you please raise an MR for the same?

๐Ÿ‡ฎ๐Ÿ‡ณIndia kunal_singh

Looks like the issue is already fixed in the 1.0.x-dev. We just have to publish a new release.

๐Ÿ‡ฎ๐Ÿ‡ณIndia kunal_singh

Merged!!

Thanks for the support.

๐Ÿ‡ฎ๐Ÿ‡ณIndia kunal_singh

Apologies for the multiple changes. The Pipeline was not configured. I did that and things are working fine now.

Thanks @mukesh88 & @kenyoOwen for the contribution.

๐Ÿ‡ฎ๐Ÿ‡ณIndia kunal_singh

kunal_singh โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia kunal_singh

I would love to contrib in the migration. Also If possible please add me as a maintainer.

๐Ÿ‡ฎ๐Ÿ‡ณIndia kunal_singh

Closing this issue because we have removed the support of D8 from this module.

๐Ÿ‡ฎ๐Ÿ‡ณIndia kunal_singh

MR Merged!!

Thank you everyone for the contribution.

Production build 0.69.0 2024