- Merge request !30Issue #3294846: Expose the url to the entity for Views β (Merged) created by devkinetic
- π¦πΊAustralia imclean Tasmania
This works well and is a valuable addition to the module. We use Views Data Export for a lot of our clients and being able to export the data for processing makes this even more useful.
- First commit to issue fork.
- π³π΄Norway eiriksm Norway
Does that mean you have tested and reviewed the code, and as such consider the addition reviewed and tested by the community? π€
- π¦πΊAustralia imclean Tasmania
I've tested and reviewed the code and it does what it claims to do. Although I alone am not the community, if you're happy for a single review then I'll update the status.
- πΊπΈUnited States devkinetic
I've been using my patch in production for 2 years with no issues, so I certainly support merging this in. A welcome addition before merging would be updating the linkchecker report to include this field by default instead of the existing entity link. This may require an update hook for existing installs
-
eiriksm β
committed c44fad89 on 8.x-1.x authored by
devkinetic β
Issue #3294846 by devkinetic, eiriksm, imclean: Expose the url to the...
-
eiriksm β
committed c44fad89 on 8.x-1.x authored by
devkinetic β
-
eiriksm β
committed 43b8d0da on cherry-pick-c44fad89
Issue #3294846 by devkinetic, eiriksm, imclean: Expose the url to the...
-
eiriksm β
committed 43b8d0da on cherry-pick-c44fad89
-
eiriksm β
committed 9a40f006 on 2.0.x
Issue #3294846 by devkinetic, eiriksm, imclean: Expose the url to the...
-
eiriksm β
committed 9a40f006 on 2.0.x
-
eiriksm β
committed 38307cf4 on cherry-pick-9a40f006
Issue #3294846 by devkinetic, eiriksm, imclean: Expose the url to the...
-
eiriksm β
committed 38307cf4 on cherry-pick-9a40f006
-
eiriksm β
committed bc8b841a on 2.1.x
Issue #3294846 by devkinetic, eiriksm, imclean: Expose the url to the...
-
eiriksm β
committed bc8b841a on 2.1.x
Automatically closed - issue fixed for 2 weeks with no activity.