Open NathanDayFE opened 1 year ago
@avstudnitz Any chance of having this PR reviewed?
@avstudnitz Apologies for double tagging but is there any chance of having this PR reviewed and merged?
Hello, future search engine result people such as myself! In case you get to this result via some form of a Magento error in management about Phrase and getText() as a string, know that this patch can fix your issue until @avstudnitz merges in this commit or fixes it another way. Here's how I solved it:
composer.json
file by adding the patch repository to the repositories section:
"repositories": {
"avsscopehintpatch": {
"type": "git",
"url": "https://github.com/NathanDayFE/AvS_ScopeHint2"
},
...any other repos here
}
"require": {
"avstudnitz/scopehint2": "dev-comment-phrase#cb2cfa4",
....any other packages here
}
ℹ️ Note: The
dev-
in the package version is composer magic to let it know to use the branch instead of a release number, and the#cb2cfa4
is the hash of the specific commit in the new repo. The commit is not required, but I like to use it for specificity, and so you don't have to usecomposer clear-cache
when new commits are added.
composer upgrade avstudnitz/scopehint2
bin/magento setup:upgrade
to finalizeThanks again @NathanDayFE for the work on this patch!
resolves #33