Skip to content
This repository has been archived by the owner on Aug 16, 2024. It is now read-only.

fix: Fix url helper calls in 2 templates #28

Merged
merged 3 commits into from
Jan 26, 2024
Merged

Conversation

fibble
Copy link
Contributor

@fibble fibble commented Jan 25, 2024

Description

Calls url helper method properly in 2 refactored templates.

Related issue: VOL-4874

Before submitting (or marking as "ready for review")

  • Does the pull request title follow the conventional commit specification?
  • Have you performed a self-review of the code
  • Have you have added tests that prove the fix or feature is effective and working
  • Did you make sure to update any documentation relating to this change?

ilindsay
ilindsay previously approved these changes Jan 25, 2024
JoshuaLicense
JoshuaLicense previously approved these changes Jan 25, 2024
@fibble fibble dismissed stale reviews from JoshuaLicense and ilindsay via 5dc7975 January 26, 2024 10:20
@JoshuaLicense JoshuaLicense merged commit ea99391 into main Jan 26, 2024
3 of 6 checks passed
@JoshuaLicense JoshuaLicense deleted the fix-url-helper-call branch January 26, 2024 10:57
jerotire pushed a commit to dvsa/vol-app that referenced this pull request Aug 16, 2024
* fix: Other template fixes where getHelperPluginManager was being used causing errors

* fix: Corrects usage of url helper in 2 refactored templates. Erroneous call to fromRoute removed. Tested ok.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants