- Issue created by @bluegeek9
- Status changed to Needs review
over 1 year ago 8:52am 9 September 2023 - Open on Drupal.org →Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7last update
over 1 year ago Waiting for branch to pass - 🇮🇳India radhe_shyam
hello bluegeek9
I solved the issue
I changed parent: system.admin_config_system of Visitors.setting
and removed
-visitors.index:
- title: 'Visitors'
- route_name: visitors.index
- parent: user.admin_index The last submitted patch, 2: 3376392.patch, failed testing. View results →
- last update
over 1 year ago 61 pass - Status changed to RTBC
over 1 year ago 2:17pm 11 September 2023 - 🇮🇳India roshni27
The above patch applied cleanly, and it is working as expected, as indicated by the screenshot. I recommend moving it to "RTBC +1."
- Status changed to Needs work
over 1 year ago 3:05pm 11 September 2023 - 🇺🇸United States bluegeek9
Hi @radhe_shyam,
Thank you for your contribution to the project.
Removing visitors.index and visitors.routes is not part of the request. Additionally, visitors.settings was updated.
- Status changed to Needs review
over 1 year ago 8:59am 12 September 2023 - Open on Drupal.org →Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7last update
over 1 year ago Waiting for branch to pass - 🇮🇳India roshni27
Hi @bluegeek9 Please review the patch, and I have added descriptions as well. If you do not want them, please let me know, and I will update the patch accordingly.
- last update
over 1 year ago 61 pass - Merge request !121Issue #3376392 by bluegeek9: Move Visitors Report menu → (Merged) created by bluegeek9
-
bluegeek9 →
committed 891e4759 on 8.x-2.x
Issue #3376392: Move Visitors Report menu
-
bluegeek9 →
committed 891e4759 on 8.x-2.x
- Status changed to Fixed
7 months ago 7:44pm 17 May 2024 - 🇺🇸United States bluegeek9
This issue was resolved. A new release, 8.x-2.17, will be made soon, May 31st.
This is an excellent opportunity to beta test the dev branch, and report any issues.
Contributors (5)
bluegeek9 → , roshni27 → , abhishek_gupta1 → , sarwan_verma → , SandeepSingh199 →
Changelog
Issues: 32 issues resolved.
Changes since 8.x-2.16 → :
Bug
- #3432630 → : Remove Not rendering HTML
- #3395217 → by bluegeek9 → : On a fresh install of a Drupal site with this module included, we get dependency errors
- #3400985 → by bluegeek9 → : Site break when placed visitors module
- #3413155 → by abhishek_gupta1 → , bluegeek9 → : Give Default value to all $agent keys
- #3383142 → by bluegeek9 → : ParseError: syntax error, unexpected
- #3380760 → : Warning: Undefined array key
Feature
- #3376256 → : Page Performance Metrics
- #3378568 → : Ajax Replace Report
- #3376234 → : Route reports
- #3378580 → : Browser Report(s)
- #1087268 → by bluegeek9 → : Exclude robots from the statistics
- #3376233 → by bluegeek9 → : View filter date range
- #3369318 → : Log OS and device infomation - BrowserCap replacement
- #3376235 → by bluegeek9 → : Drush command: Download MaxMind database
- #3250285 → by bluegeek9 → : Performance: Add db indexes
Task
- #3389685 → : Three different errors displayed
- #3376397 → by bluegeek9 → , roshni27 → : Remove block settings from visitors.config
- #3376392 → : Move Visitors Report menu
- #3444385 → : Module path
- #3443031 → by bluegeek9 → : Issues reported by PHPCS
- #3392006 → by bluegeek9 → , sarwan_verma → : watchdog_exception() deprecated
- #3423013 → : Deprecated function user_role_names()
- #3423001 → by bluegeek9 → : composer.json missing "repositories"
- #3397326 → by bluegeek9 → , SandeepSingh199 → : Issues reported by PHPStan
- #3401384 → : Remove Drupal 9 from GitLab CI
- #3393046 → by bluegeek9 → : Code Coverage
- #3377961 → : Replace visitors/hosts reports with views
- #3377962 → : Replace visitors/hits with Views
- #3377964 → : Replace /visitors/pages with Views
- #3377960 → by bluegeek9 → : Log visitor local time
- #3376241 → : Drush Command: Rebuild Geo Location
- #3377958 → : Tracking cookies
Automatically closed - issue fixed for 2 weeks with no activity.