fix typos and link the doc Replacing Component Helper

DEPRECATIONS.md

@@ -2,7 +2,7 @@
 
 ## ember-promise-modals.modals-from-string
 
-_Added in 3.1.0, functionality to be removed in 4.0.0_
+_Added in 3.1.0, functionality to be removed in 5.0.0_
 
 The new Embroider build system prefers static imports when dynamically invoking components. To resolve this deprecation, you need to replace the path to the modal component with an imported component class:
 
@@ -30,4 +30,6 @@ export class ApplicationController extends Controller {
 }
 ```
 
-This will not work with components organized in the (unsupported) _pods_ structure. If you are using pods, you will need to migrate your component to either the classic structure or to component co-location.
+> Caution: old-style components that have their template in `app/templates/components` instead of co-located next to their Javascript in `app/components` can't work correctly when discovered via their component class, because there's no way to locate the template. They should either port to being co-located (which is a simple mechanical transformation and highly recommended) or should import their own template and set it as layout as was traditional in addons before co-location was available.
+
+-- [Replacing Component Helper.md](https://github.com/embroider-build/embroider/blob/main/docs/replacing-component-helper.md#when-youre-invoking-a-component-youve-been-given)

addon/modal.js

@@ -13,7 +13,7 @@ import { defer } from 'rsvp';
 /**
  * @class Modal
  * @param {Service} service
- * @param {String} name
+ * @param {String} componentClass
  * @param {any} data
  * @param {ModalOptions} options
  * @method close - closes the modal

addon/services/modals.js

@@ -42,7 +42,7 @@ export default Service.extend({
    */
   open(componentClass, data, options) {
     deprecate(
-      `Ember Promise Modals: For extendend compatibility with Embroider and it's tree-shaking ability, you need to import and pass in the component class of the modal component you want to display instead of "${componentClass}".`,
+      `Ember Promise Modals: For extended compatibility with Embroider and its tree-shaking ability, you need to import and pass in the component class of the modal component you want to display instead of "${componentClass}".`,
       typeof componentClass !== 'string',
       {
         id: 'ember-promise-modals.modals-from-string',

tests/application/deprecations-test.js

@@ -11,7 +11,7 @@ module('Application | Deprecations', function (hooks) {
   setupPromiseModals(hooks);
 
   if (!ENV.SKIP_EMBROIDER_DEPRECATION_TEST) {
-    test("opening a modal by it's path triggers a deprecation", async function (assert) {
+    test('opening a modal by its path triggers a deprecation', async function (assert) {
       await visit('/');
 
       assert.dom('.epm-backdrop').doesNotExist();