Bhopal
Account created on 9 September 2023, over 1 year ago
#

Merge Requests

More

Recent comments

🇮🇳India zaryab_drupal Bhopal

@mornel The version you mentioned is not available for Drupal 11, I am attaching screenshot and URL of Drupal.org https://www.drupal.org/project/ckeditor5_markdown_editor/releases/1.0.x-dev

This below version is available for Drupal 11 and it successfully install in D11.
https://www.drupal.org/project/ckeditor5_markdown_editor/releases/1.0.0-...

🇮🇳India zaryab_drupal Bhopal

zaryab_drupal made their first commit to this issue’s fork.

🇮🇳India zaryab_drupal Bhopal

zaryab_drupal made their first commit to this issue’s fork.

🇮🇳India zaryab_drupal Bhopal

zaryab_drupal changed the visibility of the branch 3486349-update-php-version to hidden.

🇮🇳India zaryab_drupal Bhopal

Working as Expected, Please see attached screenshot.

🇮🇳India zaryab_drupal Bhopal

You're correct. By avoiding the use of '#type' => 'path', you enable users to enter relative paths such as /abc rather than requiring a complete URL like https://www.example.com/abc. Instead, using '#type' => 'textfield' provides greater flexibility and aligns more closely with Drupal's standard handling of internal paths.

🇮🇳India zaryab_drupal Bhopal

Updated the RedirectAfterRegistrationConfigForm to implement proper dependency injection for the path validator service. Added validation for the destination field to ensure it is a valid internal path.

🇮🇳India zaryab_drupal Bhopal

Updated code with this below :

if (isset($form_copy['subject'])) {
$form['subject'] = $form_copy['subject'];
}

The use of the @ symbol in PHP is generally discouraged because it suppresses errors, which can make debugging difficult. Instead, we should use proper checks like isset() or empty() to safely handle variables.

🇮🇳India zaryab_drupal Bhopal

zaryab_drupal made their first commit to this issue’s fork.

🇮🇳India zaryab_drupal Bhopal

Updated the Core Version Requirement

🇮🇳India zaryab_drupal Bhopal

zaryab_drupal made their first commit to this issue’s fork.

🇮🇳India zaryab_drupal Bhopal

zaryab_drupal made their first commit to this issue’s fork.

🇮🇳India zaryab_drupal Bhopal

zaryab_drupal made their first commit to this issue’s fork.

🇮🇳India zaryab_drupal Bhopal

Working as expected.
See attached screenshot.

🇮🇳India zaryab_drupal Bhopal

zaryab_drupal made their first commit to this issue’s fork.

🇮🇳India zaryab_drupal Bhopal

zaryab_drupal made their first commit to this issue’s fork.

🇮🇳India zaryab_drupal Bhopal

zaryab_drupal made their first commit to this issue’s fork.

🇮🇳India zaryab_drupal Bhopal

zaryab_drupal made their first commit to this issue’s fork.

🇮🇳India zaryab_drupal Bhopal

zaryab_drupal made their first commit to this issue’s fork.

🇮🇳India zaryab_drupal Bhopal

zaryab_drupal made their first commit to this issue’s fork.

🇮🇳India zaryab_drupal Bhopal

zaryab_drupal made their first commit to this issue’s fork.

🇮🇳India zaryab_drupal Bhopal

zaryab_drupal made their first commit to this issue’s fork.

🇮🇳India zaryab_drupal Bhopal

zaryab_drupal made their first commit to this issue’s fork.

🇮🇳India zaryab_drupal Bhopal

Added readme.md file to the code base in accordance to readme.txt

Please review and suggest if any other change required,
Moving the issue to needs review.

🇮🇳India zaryab_drupal Bhopal

zaryab_drupal made their first commit to this issue’s fork.

🇮🇳India zaryab_drupal Bhopal

zaryab_drupal changed the visibility of the branch 3399956-tab-highlights-for to hidden.

Production build 0.71.5 2024