Boston
Account created on 19 April 2018, almost 7 years ago
#

Merge Requests

Recent comments

πŸ‡ΊπŸ‡ΈUnited States avattikonda Boston

Patch #4 works for me using Drupal 10.3.1

πŸ‡ΊπŸ‡ΈUnited States avattikonda Boston

Would it be possible to release a new version of the module that supports Drupal 11? We greatly appreciate the work you've done so far, and having Drupal 11 compatibility would help us and other users continue to benefit from the module in our projects since the Rector patch applied cleanly and after running Upgrade Status module it shows are Drupal 11 compatible.

πŸ‡ΊπŸ‡ΈUnited States avattikonda Boston

Code changes look good to me. Tested on fresh Drupal 10.4.1 installation.

πŸ‡ΊπŸ‡ΈUnited States avattikonda Boston

@ramu_bharu

I tested the installation of the vrssfa module and did encounter above errors or warnings.

Below are the steps I followed to replicate

I attempted to enable the vrssfa module after downloading it via Composer and performing a fresh installation of Drupal 10.4. However, when I run drush en vrssfa, I encounter the following error:

In Renderer.php line 303:

Render context is empty, because render() was called outside of a renderRoot() or renderInIsolation() call. Use renderInIsolation()/renderRoot() or #lazy_builder/#pre_render instead.

I had attached a patch version which will fix the above error

Here I used renderInIsolation(): which safely renders the output outside of a regular render context, which is necessary when we're rendering something outside the normal page build process (like in an install hook).

πŸ‡ΊπŸ‡ΈUnited States avattikonda Boston

Hello,

Downgrade of doctrine/depreciations to 1.1.3 in #14 resulted in the following warnings

Warning: include(): Failed opening '/var/www/vendor/composer/../doctrine/deprecations/lib/Doctrine/Deprecations/Deprecation.php' for inclusion (include_path='/var/www/vendor/pear/archive_tar:/var/www/vendor/pear/console_getopt:/var/www/vendor/pear/pear-core-minimal/src:/var/www/vendor/pear/pear_exception:.:/usr/local/lib/php') in include() (line 576 of /var/www/vendor/composer/ClassLoader.php).

After restarting the local server, the warnings were resolved on my end.

πŸ‡ΊπŸ‡ΈUnited States avattikonda Boston

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

Production build 0.71.5 2024