Hello community,
Looks like I found the place where we can fix the problem.
For some reason $request
object in /core/lib/Drupal/Core/Pager/PagerParameters.php:65
does not contain 'page'
property on AJAX requests, that's why we always have current page equals to 0.
The solution is simple, just add a check if the current request is an AJAX request and try to get the 'page' value from the request URI.
Please test my patch, hope it helps.
driverok → credited ysamoylenko → .
The changes have been merged from another MR in 💬 Drupal 10 Fixed
Please use the 3.0.0-alpha1 release for Drupal 10.
The new module release will be created soon.
Hello @dan612. I propose using the latest module version.
It does not require importing places from CSV files and uses https://met.no API.
To add a new place please follow the instructions described in the updated README.md file for 3.x module version.
https://git.drupalcode.org/project/weather/-/blob/3.x/README.md?ref_type...
The further module development will be in the 3.x branch.
The changes look good to me and will be merged soon. Thank you all for your contribution.
Thank you all for your contribution the changes will be merged soon.
+
driverok → credited ysamoylenko → .
driverok → credited ysamoylenko → .
Thank you all for your contribution. The MR will be merged into the dev branch soon.
ysamoylenko → made their first commit to this issue’s fork.
The changes were applied to the latest module dev version. Thanks for your contribution.
@gun2dru Thank you for your MR and for fixing the deprecation issues. The changes will be merged soon.
ysamoylenko → made their first commit to this issue’s fork.
@gun2dru thank you for providing your test results.
You are right, starting from the module version configuration schema of the module has changed.
As I can see the issue was created on 9 Apr 2019, so a lot of changes were applied to the module from that time.
If you are using an outdated version of the module please create a DB backup before the module upgrade, then pull a new version and run "drush updb", all outdated configurations will be upgraded to the new configuration schema format.
The changes were merged. Thanks for your contribution @snap_x.
Thank you for publishing your MR. The changes will be merged soon.
The patch has been committed.
Module lines have differences since the patch was published but it fixes the deprecations. It will be committed soon.
ysamoylenko → created an issue.
The module is already compatible with Drupal Core 8.8 -> 10 starting from 2.x-dev release.
This issue has been already fixed in the last module release.
The changes have been committed.
Thank you for the prepared MR, the changes will be merged soon.
Thank you for the prepared MR and testing. The changes are already applied in the 📌 Fix the issues reported by phpcs Fixed .
The patch has been committed.
Thank you for implementing and testing the patch. The changes will be committed soon.
The changes have been committed.
Thank you for the patch implementation and review activities. The changes will be committed soon.
The patch has been committed.
Thank you for implementing and reviewing this patch. It will be committed soon.
driverok → credited ysamoylenko → .
driverok → credited ysamoylenko → .
Thank you for your efforts in preparing the fixes and review. The changes have been committed.
Hello, I fixed DI issues in my patch. Please review.
The changes look good to me. Please commit them.
The fix for this problem is already available in a release 2.0.9 →
Please try to re-import places again.
Thank you for reporting the issue.
Thank you for your efforts spent making this patch.
The patch from 🐛 yr.no/met.no XML api will be closed at beginning of 2021 Fixed has been committed.
The changes are published in a release 2.0.9 →
Starting from 2.1 we are using an XML version of the forecast from the met.no API.
The patch has been committed.
The changes are published in a release 2.0.9 →
Starting from 2.1 we are using an XML version of the forecast from the met.no API.
Unit tests still require to be updated.
Thanks for the patch for 2.0.x branch.
It will be applied soon. It will also contain the fix for 🐛 This URL is no longer valid Fixed
Closing the issue as the problem has been already resolved in the mentioned issues.
The MR code looks good to me. The changes will be merged soon.
ysamoylenko → made their first commit to this issue’s fork.
The changes have been committed.
Thank you for your patch. It will be committed soon.
ysamoylenko → created an issue.
The changes have been committed.
The patch looks good to me. Thanks for implementing it.
Thank you for reviewing the patch. It has been committed.
The patch has been committed. A new release will be created soon.
Thank you for your patch. It looks good to me.
The patch has been committed. A new release will be created soon.
Hello @c.moisiadis,
The patch looks good to me.
It will be committed soon.
Thanks everyone for testing and reporting the issue.
ysamoylenko → created an issue.
Hello @WannaAlwaysBe,
Thank you for the patch, it looks good to me.
The patch will be committed soon.
Hello @Rar9, @joshuautley,
This is a feature request, as imagefield_tokens module has no integration with colorbox module image formatter.
However, I've implemented the required plugin for you.
Please test it as a patch first.
To use this new functionality please use ImageField Tokens Widget and ImageField Tokens: Colorbox formatted.
Input:
Output:
So, please review my patch and let me know about the results.
Confirming that https://www.drupal.org/project/drupal/issues/1149078#comment-14271390 🐛 States API doesn't work with multiple select fields Needs review works correctly with empty values on 9.5.3.
Hello @Berdir,
Thank you for your response.
These array keys do not exist in the $info
array.
I am adding a new patch with a non-empty evaluation.
According to the mentioned issue, it looks similar but for another section of the report.
In our environment, I have not faced it.
driverok → credited ysamoylenko → .
driverok → credited ysamoylenko → .
Hello @savel,
Thank you for the patch.
It looks good to me.
Happy to RTBC!
The changes have been committed.
Hello @danflanagan8 and @yashingole,
Thank you for the patch and review. The changes look good to me.
Thank you for your contribution.
The changes will be merged soon.
The changes have been committed.
The changes look good to me.
They will be merged soon.
Hello,
Please try to upgrade the module to the latest version, it contains a lot of known issues bug fixes.
https://www.drupal.org/project/multiple_registration/releases/3.2.0 →
Hello,
Please try to upgrade the module to the latest version, it contains a lot of known issues bug fixes.
https://www.drupal.org/project/multiple_registration/releases/3.2.0 →
Hello @Cheope,
Please try to upgrade the module to the latest version, it contains a lot of known issues bug fixes.
https://www.drupal.org/project/multiple_registration/releases/3.2.0 →
The patch has been committed.
The changes look good and will be committed soon.
The changes have been merged. Thank you for your contribution.
Hello @Foxy-vikvik,
The changes look good. Happy to RTBC.
ysamoylenko → made their first commit to this issue’s fork.