Create the structure of a module (.info, .yaml, block object)

Remember, you need to set the issue to a status of "Needs Review"

1. +++ b/web/modules/custom/commerce_add_to_cart_confirmation/commerce_add_to_cart_confirmation.info.yml
   @@ -0,0 +1,6 @@
   +  - commerce
   

   Will need a dependency on commerce_order and commerce_cart (confirm those are the right spellings)

2. +++ b/web/modules/custom/commerce_add_to_cart_confirmation/commerce_add_to_cart_confirmation.info.yml
   @@ -0,0 +1,6 @@
   \ No newline at end of file
   

   Missing the newline at the end of the info.yml file

3. +++ b/web/modules/custom/commerce_add_to_cart_confirmation/commerce_add_to_cart_confirmation.services.yml
   @@ -0,0 +1,6 @@
   +services:
   +  commerce_add_to_cart_confirmation.add_to_cart_listener:
   +    class: Drupal\commerce_add_to_cart_confirmation\EventSubscription\AddToCartListener
   +    arguments: []
   +    tags:
   +      - { name: event_subscriber }
   

   This should be a part of another patch, not the "simple structure" portion.

4. +++ b/web/modules/custom/commerce_add_to_cart_confirmation/src/Controller/AddToCartConfirmation.php
   @@ -0,0 +1,17 @@
   +/**
   + * Created by PhpStorm.
   + * User: saurabh
   + * Date: 21/6/17
   + * Time: 1:21 AM
   + */
   

   This comment doesn't follow Drupal PHP Code Standards.

+++ b/web/modules/custom/commerce_add_to_cart_confirmation/commerce_add_to_cart_commerce.permissions.yml
@@ -0,0 +1,2 @@
+  title: View Add to Cart confirmation

This is a nitpick but 'confirmation' should be capitalized as well since all the other words are.

+++ b/web/modules/custom/commerce_add_to_cart_confirmation/src/Controller/AddToCartConfirmation.php
@@ -0,0 +1,17 @@
+<?php
+
+namespace Drupal\commerce_add_to_cart_confirmation\Controller;
+
+use Drupal\Core\Controller\ControllerBase;
+
+class AddToCartConfirmation extends ControllerBase
+{
+
+  /**
+   * Purpose of this function is to create popup modal.
+   */
+  public function createPopup() {
+    echo 'Hello world';
+  }
+
+}

We dont want that from this patch. But I heard from you it's already moved.