Dev by TheStaches · Pull Request #61 · OriginCodeAcademy/origin-calendar

TheStaches · 2018-10-05T23:53:29Z

No description provided.

* adds multiple admin accounts * corrects variables * changes bootscript

* sets up remove apt button * sends email to user/admin

* changes apt request approve function * changes aptRequests

* changes apt request approve function * changes aptRequests * changes time to display correctly in admin availability

* saving current stuff * made individual instructor pages and created e2e tests * made some changes to the circle config * changed start script * changed start script * added run build in the test script * imported moment package * made changes to nightmare test, created a student user, made changes individaul instructors components * added nightmare tests and deletes requested options from state * saving current stuff * made individual instructor pages and created e2e tests * made some changes to the circle config * changed start script * changed start script * added run build in the test script * imported moment package * made changes to nightmare test, created a student user, made changes individaul instructors components * added nightmare tests and deletes requested options from state

* adds multiple admin accounts * corrects variables * changes bootscript * Adds remove appointment on pending apppointment * adds instructor availability delete functionality * Deletes Slot and send email to pending request students * Fixes delete button * fixes delete button * fixes merge conflicts * adds instructor availability delete functionality * fixes merge conflict * Fixes delete button * fixes merge conflict * Adds remove appointment on pending apppointment * adds instructor availability delete functionality * Deletes Slot and send email to pending request students * Fixes delete button * fixes delete button * fixes merge conflicts * adds instructor availability delete functionality * fixes merge conflict * fixes instructor names * fixes merge conflict

* adds multiple admin accounts * corrects variables * changes bootscript * sets up remove apt button * sends email to user/admin * trash icon * new button * sets up remove apt button * sends email to user/admin * trash icon * new button

* adds multiple admin accounts * corrects variables * changes bootscript * Sends instructor email on appointment request * Setting up google calendar * Merging conflicts * Merging conflicts * Routing api code to remote method * Routing api code to remote method * adds token to user model * Adds token to user model * Gets auth token for admin when logging in * fixes authorize * sets auth tokens to user models and refreshes token on login * Sends instructor email on appointment request * Setting up google calendar * Merging conflicts * Merging conflicts * Routing api code to remote method * Routing api code to remote method * adds token to user model * Adds token to user model * Gets auth token for admin when logging in * fixes authorize * sets auth tokens to user models and refreshes token on login * fixes merge conflicts * fixes merge conflicts * changes approve function and deletes extra googleAuth * Fixes emailAdmin on pending appointment * fixes rendering pending appointments * fixes merge issues * fixes merge issues * Merging conflicts * fixes merge issues * Routing api code to remote method * fixes merge issues * fixes merge issues * fixes merge issues * Gets auth token for admin when logging in * fixes authorize * fixes merge issues * fixes merge issues * Merging conflicts * Merging conflicts * fixes merge issues * fixes merge issues * fixes merge issues * fixes merge issues * fixes merge issues * fixes merge issues * sets auth tokens to user models and refreshes token on login * fixes merge issues * changes approve function and deletes extra googleAuth * fixes emailAdmin * fixes pendingAppointments

* removed confirmed apts * removed confirmed apts

qlty-cloud-legacy · 2018-10-05T23:53:48Z

server/models/booked-apt.js

+      };
+
+      BookedApt.remoteMethod('removedConfirmed', {
+        accepts: [{arg: 'email', type: 'string', required: true},


Similar blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-05T23:53:48Z

server/models/apt-request.js

+  };
+
+  Aptrequest.remoteMethod('emailAdmin', {
+    accepts: [{arg: 'instructorEmail', type: 'string', required: true},


Similar blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-05T23:53:48Z

src/components/Appointments/Pending.jsx

                  <td>{e.issueDescription}</td>
                  <td>{moment(e.time).format('L')}</td>
                  <td>{moment(e.time).format('hh:mm a')}</td>
+                  <td>


Similar blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-05T23:53:48Z

src/components/Appointments/index.jsx

+                      <td>{moment(e.timeSlot).format('L')}</td>
+                      <td>{moment(e.timeSlot).format('hh:mm a')}</td>
+                      <td>{e.duration}</td>
+                      <td>


Similar blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-05T23:53:48Z

server/models/visitor.js

+    });
+  };
+
+  Visitor.remoteMethod('oAuthConfirm', {


Similar blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-05T23:53:49Z

src/components/Request/Christianview.jsx

+        const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+        const time = this.state.time;
+        const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+        axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 4 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-05T23:53:49Z

src/components/Request/Anthonyview.jsx

+        const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+        const time = this.state.time;
+        const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+        axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 4 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-05T23:53:49Z

src/components/Request/index.jsx

+    const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+    const time = this.state.time;
+    const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+    axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 4 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-05T23:53:50Z

src/components/Request/Anthonyview.jsx

+        })
+      }
+
+      handleTimeSlot(e) {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-05T23:53:50Z

server/boot/create-admin.js

+          });
+      });
+  });
+  Visitor.findOrCreate({


Similar blocks of code found in 4 locations. Consider refactoring.

anthonyvalera · 2018-10-08T18:33:16Z

client/googlef392b883fe8ce677.html

@@ -0,0 +1 @@
+google-site-verification: googlef392b883fe8ce677.html


Remove this file

anthonyvalera · 2018-10-08T19:59:15Z

server/boot/create-admin.js

+          });
+      });
+  });
+  Visitor.findOrCreate({


These functions should be condensed into some sort of loop

anthonyvalera · 2018-10-08T20:07:39Z

src/components/App/index.jsx

          { this.state.user && (this.state.isAdmin === false) ?
            <li className="nav-item">
-              <Link to="/request">Request</Link>
+              <Link id='reqtest' to="/request" >Request</Link>


Is this supposed to be request?

anthonyvalera · 2018-10-08T21:07:37Z

src/components/AdminAvailability/index.jsx

+        .then(res => {
+            res.data.map(request => {
+                axios.delete(`/api/AptRequests/${request.id}`)
+                axios.post(`/api/AptRequests/denyEmail`, {


Posting the deny email should come after the DELETE request has been fulfilled. Throw this POST request within a .then of the previous request

anthonyvalera · 2018-10-08T21:29:46Z

src/components/Appointments/Pending.jsx

+      time: time,
+    })
+      .then(function (response) {
+        console.log(response);


Remove superfluous console.log(s) from client side

anthonyvalera · 2018-10-08T22:33:39Z

src/components/App/index.jsx

 import SignIn from '../SignIn';
 import Request from '../Request';
 import Availability from '../AdminAvailability';
+import Christianviews from '../Request/Christianview';


These files are essentially the same. Condense the three components into one and pass down differences through props

anthonyvalera · 2018-10-08T22:35:33Z

src/components/Request/index.jsx

+    const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+    const time = this.state.time;
+    const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+    axios.post(`/api/AptRequests/emailAdmin`, {


This request should go inside the .then of the previous axios.post. We should expect that the request was successful before getting an email about it

anthonyvalera · 2018-10-08T22:35:44Z

src/components/Request/index.jsx

+      time: time,
+      studentName: studentName,  
+    })
+      .then()


Remove this .then

anthonyvalera · 2018-10-08T22:38:20Z

tests/e2e/main.spec.js

+chai.use(chaiHttp);
+const expect = chai.expect;
+
+const path = 'http://localhost:8080' || process.env.PORT;


This string will always be truthy, so it will never take process.env.PORT (which isn't needed in your test). Change this line to just the string

anthonyvalera · 2018-10-08T22:39:12Z

tests/main.spec.js

 chai.use(chaiHttp);
 const expect = chai.expect;

+const url = 'http://localhost:8080' || process.env.PORT;


This line isn't needed

qlty-cloud-legacy · 2018-10-08T23:55:37Z

src/components/Request/Anthonyview.jsx

+import moment from 'moment';
+
+export default class Anthonyview extends react.Component{
+    constructor(props){


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-08T23:55:37Z

src/components/Request/Michaelview.jsx

+        const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+        const time = this.state.time;
+        const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+        axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-08T23:55:37Z

src/components/Request/Christianview.jsx

+        const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+        const time = this.state.time;
+        const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+        axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-08T23:55:37Z

src/components/Request/Michaelview.jsx

+      }
+
+      handleSubmit(e) {
+        axios.post(`/api/Visitors/${this.props.user.id}/aptRequests`, {


Similar blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-08T23:55:37Z

src/components/Request/Christianview.jsx

+      }
+
+      handleSubmit(e) {
+        axios.post(`/api/Visitors/${this.props.user.id}/aptRequests`, {


Similar blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-08T23:55:38Z

src/components/Request/Anthonyview.jsx

+          })
+      }
+
+      handleSubmit(e) {


Function handleSubmit has 41 lines of code (exceeds 25 allowed). Consider refactoring.

qlty-cloud-legacy · 2018-10-08T23:55:38Z

src/components/Request/Christianview.jsx

+
+
+
+    render(){


Function render has 27 lines of code (exceeds 25 allowed). Consider refactoring.

qlty-cloud-legacy · 2018-10-08T23:55:38Z

src/components/Request/Christianview.jsx

+
+      }
+
+      handleSubmit(e) {


Function handleSubmit has 40 lines of code (exceeds 25 allowed). Consider refactoring.

qlty-cloud-legacy · 2018-10-08T23:55:38Z

src/components/Request/Anthonyview.jsx

+          "slotId": this.state.slotId,
+          "instructorId": "anthony"
+        }).then((response) => {
+          axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-08T23:55:38Z

src/components/Request/index.jsx

+    const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+    const time = this.state.time;
+    const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+    axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:30:16Z

src/components/Request/Anthonyview.jsx

+import moment from 'moment';
+
+export default class Anthonyview extends react.Component{
+    constructor(props){


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:30:16Z

src/components/Request/Michaelview.jsx

+        const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+        const time = this.state.time;
+        const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+        axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:30:16Z

src/components/Request/Christianview.jsx

+        const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+        const time = this.state.time;
+        const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+        axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:30:16Z

src/components/Request/Michaelview.jsx

+      }
+
+      handleSubmit(e) {
+        axios.post(`/api/Visitors/${this.props.user.id}/aptRequests`, {


Similar blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:30:16Z

src/components/Request/Christianview.jsx

+      }
+
+      handleSubmit(e) {
+        axios.post(`/api/Visitors/${this.props.user.id}/aptRequests`, {


Similar blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:30:17Z

src/components/Request/Anthonyview.jsx

+          })
+      }
+
+      handleSubmit(e) {


Function handleSubmit has 41 lines of code (exceeds 25 allowed). Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:30:17Z

src/components/Request/Christianview.jsx

+
+
+
+    render(){


Function render has 27 lines of code (exceeds 25 allowed). Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:30:17Z

src/components/Request/Christianview.jsx

+
+      }
+
+      handleSubmit(e) {


Function handleSubmit has 40 lines of code (exceeds 25 allowed). Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:30:17Z

src/components/Request/Anthonyview.jsx

+          "slotId": this.state.slotId,
+          "instructorId": "anthony"
+        }).then((response) => {
+          axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:30:18Z

src/components/Request/index.jsx

+    const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+    const time = this.state.time;
+    const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+    axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:46:24Z

src/components/Request/Anthonyview.jsx

+import moment from 'moment';
+
+export default class Anthonyview extends react.Component{
+    constructor(props){


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:46:24Z

src/components/Request/Michaelview.jsx

+        const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+        const time = this.state.time;
+        const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+        axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:46:24Z

src/components/Request/Christianview.jsx

+        const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+        const time = this.state.time;
+        const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+        axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:46:24Z

src/components/Request/Michaelview.jsx

+      }
+
+      handleSubmit(e) {
+        axios.post(`/api/Visitors/${this.props.user.id}/aptRequests`, {


Similar blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:46:24Z

src/components/Request/Christianview.jsx

+      }
+
+      handleSubmit(e) {
+        axios.post(`/api/Visitors/${this.props.user.id}/aptRequests`, {


Similar blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:46:25Z

src/components/Request/Anthonyview.jsx

+          })
+      }
+
+      handleSubmit(e) {


Function handleSubmit has 41 lines of code (exceeds 25 allowed). Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:46:25Z

src/components/Request/Christianview.jsx

+
+
+
+    render(){


Function render has 27 lines of code (exceeds 25 allowed). Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:46:25Z

src/components/Request/Christianview.jsx

+
+      }
+
+      handleSubmit(e) {


Function handleSubmit has 40 lines of code (exceeds 25 allowed). Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:46:26Z

src/components/Request/Anthonyview.jsx

+          "slotId": this.state.slotId,
+          "instructorId": "anthony"
+        }).then((response) => {
+          axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:46:26Z

src/components/Request/index.jsx

+    const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+    const time = this.state.time;
+    const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+    axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:50:59Z

src/components/Request/Anthonyview.jsx

+import moment from 'moment';
+
+export default class Anthonyview extends react.Component{
+    constructor(props){


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:51:00Z

src/components/Request/Michaelview.jsx

+        const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+        const time = this.state.time;
+        const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+        axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:51:00Z

src/components/Request/Christianview.jsx

+        const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+        const time = this.state.time;
+        const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+        axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:51:00Z

src/components/Request/Michaelview.jsx

+      }
+
+      handleSubmit(e) {
+        axios.post(`/api/Visitors/${this.props.user.id}/aptRequests`, {


Similar blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:51:00Z

src/components/Request/Christianview.jsx

+      }
+
+      handleSubmit(e) {
+        axios.post(`/api/Visitors/${this.props.user.id}/aptRequests`, {


Similar blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:51:01Z

src/components/Request/Anthonyview.jsx

+          })
+      }
+
+      handleSubmit(e) {


Function handleSubmit has 41 lines of code (exceeds 25 allowed). Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:51:01Z

src/components/Request/Christianview.jsx

+
+
+
+    render(){


Function render has 27 lines of code (exceeds 25 allowed). Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:51:01Z

src/components/Request/Christianview.jsx

+
+      }
+
+      handleSubmit(e) {


Function handleSubmit has 40 lines of code (exceeds 25 allowed). Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:51:01Z

src/components/Request/Anthonyview.jsx

+          "slotId": this.state.slotId,
+          "instructorId": "anthony"
+        }).then((response) => {
+          axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T00:51:01Z

src/components/Request/index.jsx

+    const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+    const time = this.state.time;
+    const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+    axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:11:37Z

src/components/Request/Anthonyview.jsx

+import moment from 'moment';
+
+export default class Anthonyview extends react.Component{
+    constructor(props){


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:11:37Z

src/components/Request/Michaelview.jsx

+        const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+        const time = this.state.time;
+        const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+        axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:11:37Z

src/components/Request/Christianview.jsx

+        const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+        const time = this.state.time;
+        const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+        axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:11:37Z

src/components/Request/Michaelview.jsx

+      }
+
+      handleSubmit(e) {
+        axios.post(`/api/Visitors/${this.props.user.id}/aptRequests`, {


Similar blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:11:37Z

src/components/Request/Christianview.jsx

+      }
+
+      handleSubmit(e) {
+        axios.post(`/api/Visitors/${this.props.user.id}/aptRequests`, {


Similar blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:11:38Z

src/components/Request/Anthonyview.jsx

+          })
+      }
+
+      handleSubmit(e) {


Function handleSubmit has 41 lines of code (exceeds 25 allowed). Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:11:38Z

src/components/Request/Christianview.jsx

+
+
+
+    render(){


Function render has 27 lines of code (exceeds 25 allowed). Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:11:38Z

src/components/Request/Christianview.jsx

+
+      }
+
+      handleSubmit(e) {


Function handleSubmit has 40 lines of code (exceeds 25 allowed). Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:11:39Z

src/components/Request/Anthonyview.jsx

+          "slotId": this.state.slotId,
+          "instructorId": "anthony"
+        }).then((response) => {
+          axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:11:39Z

src/components/Request/index.jsx

+    const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+    const time = this.state.time;
+    const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+    axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:23:19Z

src/components/Request/Anthonyview.jsx

+import moment from 'moment';
+
+export default class Anthonyview extends react.Component{
+    constructor(props){


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:23:19Z

src/components/Request/Michaelview.jsx

+        const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+        const time = this.state.time;
+        const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+        axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:23:19Z

src/components/Request/Christianview.jsx

+        const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+        const time = this.state.time;
+        const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+        axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:23:20Z

src/components/Request/Michaelview.jsx

+      }
+
+      handleSubmit(e) {
+        axios.post(`/api/Visitors/${this.props.user.id}/aptRequests`, {


Similar blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:23:20Z

src/components/Request/Christianview.jsx

+      }
+
+      handleSubmit(e) {
+        axios.post(`/api/Visitors/${this.props.user.id}/aptRequests`, {


Similar blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:23:20Z

src/components/Request/Anthonyview.jsx

+          })
+      }
+
+      handleSubmit(e) {


Function handleSubmit has 41 lines of code (exceeds 25 allowed). Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:23:20Z

src/components/Request/Christianview.jsx

+
+
+
+    render(){


Function render has 27 lines of code (exceeds 25 allowed). Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:23:20Z

src/components/Request/Christianview.jsx

+
+      }
+
+      handleSubmit(e) {


Function handleSubmit has 40 lines of code (exceeds 25 allowed). Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:23:20Z

src/components/Request/Anthonyview.jsx

+          "slotId": this.state.slotId,
+          "instructorId": "anthony"
+        }).then((response) => {
+          axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

qlty-cloud-legacy · 2018-10-09T01:23:21Z

src/components/Request/index.jsx

+    const instructorEmail = `${this.state.selectedSlot.instructorId}@origincodeacademy.com`
+    const time = this.state.time;
+    const studentName = (this.props.user.firstName + ' ' + this.props.user.lastName);
+    axios.post(`/api/AptRequests/emailAdmin`, {


Identical blocks of code found in 2 locations. Consider refactoring.

Mank858 and others added 9 commits September 27, 2018 14:13

Dje multiple admins (#45)

d166e0d

* adds multiple admin accounts * corrects variables * changes bootscript

Kw (#49)

f548cbf

* sets up remove apt button * sends email to user/admin

Dje cal 17 (#53)

30596d9

* changes apt request approve function * changes aptRequests

Dje cal 12 (#55)

81ed43d

* changes apt request approve function * changes aptRequests * changes time to display correctly in admin availability

Kw (#57)

7e40173

* adds multiple admin accounts * corrects variables * changes bootscript * sets up remove apt button * sends email to user/admin * trash icon * new button * sets up remove apt button * sends email to user/admin * trash icon * new button

removed confirmed apts (#60)

0c948a3

* removed confirmed apts * removed confirmed apts

qlty-cloud-legacy bot reviewed Oct 5, 2018

View reviewed changes

anthonyvalera suggested changes Oct 8, 2018

View reviewed changes

fixes code climate issues

4ab8b26

qlty-cloud-legacy bot reviewed Oct 8, 2018

View reviewed changes

Fixes tests

f481480

qlty-cloud-legacy bot reviewed Oct 9, 2018

View reviewed changes

Fixes tests

8ebdc51

qlty-cloud-legacy bot reviewed Oct 9, 2018

View reviewed changes

Fixes tests

3e0501f

qlty-cloud-legacy bot reviewed Oct 9, 2018

View reviewed changes

fixes testing issues

37f9cf8

qlty-cloud-legacy bot reviewed Oct 9, 2018

View reviewed changes

updating tests

3ea022d

qlty-cloud-legacy bot reviewed Oct 9, 2018

View reviewed changes

anthonyvalera approved these changes Oct 27, 2018

View reviewed changes

		@@ -0,0 +1 @@
		google-site-verification: googlef392b883fe8ce677.html No newline at end of file

Comments

Conversation

TheStaches commented Oct 5, 2018

Uh oh!

qlty-cloud-legacy bot Oct 5, 2018

Choose a reason for hiding this comment

Uh oh!

qlty-cloud-legacy bot Oct 5, 2018

Choose a reason for hiding this comment

Uh oh!

qlty-cloud-legacy bot Oct 5, 2018

Choose a reason for hiding this comment

Uh oh!

qlty-cloud-legacy bot Oct 5, 2018

Choose a reason for hiding this comment

Uh oh!

qlty-cloud-legacy bot Oct 5, 2018

Choose a reason for hiding this comment

Uh oh!

qlty-cloud-legacy bot Oct 5, 2018

Choose a reason for hiding this comment

Uh oh!

qlty-cloud-legacy bot Oct 5, 2018

Choose a reason for hiding this comment

Uh oh!

qlty-cloud-legacy bot Oct 5, 2018

Choose a reason for hiding this comment

Uh oh!

qlty-cloud-legacy bot Oct 5, 2018

Choose a reason for hiding this comment

Uh oh!

qlty-cloud-legacy bot Oct 5, 2018

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

qlty-cloud-legacy bot Oct 8, 2018

Choose a reason for hiding this comment

Uh oh!

qlty-cloud-legacy bot Oct 8, 2018

Choose a reason for hiding this comment

Uh oh!

qlty-cloud-legacy bot Oct 8, 2018

Choose a reason for hiding this comment

Uh oh!

qlty-cloud-legacy bot Oct 8, 2018

Choose a reason for hiding this comment

Uh oh!

qlty-cloud-legacy bot Oct 8, 2018

Choose a reason for hiding this comment

Uh oh!

qlty-cloud-legacy bot Oct 8, 2018

Choose a reason for hiding this comment

Uh oh!

qlty-cloud-legacy bot Oct 8, 2018

Choose a reason for hiding this comment

Uh oh!

qlty-cloud-legacy bot Oct 8, 2018

Choose a reason for hiding this comment

Uh oh!

qlty-cloud-legacy bot Oct 8, 2018

Choose a reason for hiding this comment