Code Monkey home page Code Monkey logo

ramp's Introduction

Instructions

Welcome to Ramp's frontend interview challenge.

In this challenge, you will need to fix certain bugs within the starter code provided to you.

The bugs do not depend on each other, so you can solve them independently.

You will submit a CodeSandbox link with your response.

Prerequisites

Coding

Since you need to submit a CodeSandbox link with your response (See Submission), we recommend that you create the CodeSandbox first, solve the bugs in your generated CodeSandbox, and then share the link with us. You can also work locally first, and upload at the end.

Upload the project to CodeSandbox

NOTE: We recommend you use this method to upload your project (with the CLI) rather than importing directly from Github to generate a CodeSandbox.

  • Run yarn install or npm install
  • Run yarn upload or npm run upload
  • If this is the first time using CodeSandbox CLI, it will ask you to log in with Github first
  • You might be prompted: We will upload XXX static files to your CodeSandbox upload storage and then a list of files (typically DS_Store or desktop.ini files). It's fine if you upload with these, or you can manually remove them before uploading.
  • Confirm that you want to proceed with deployment
  • Once it finishes, you will get the link for your CodeSandbox. Also, you can log in to the website with your Github account and see your projects to retrieve the link.
  • Start working directly on the CodeSandbox

Reference: https://codesandbox.io/docs/importing#import-local-projects-via-cli

Or

Run the server locally

  • Run yarn install or npm install
  • Run yarn start
  • The server will be available in http://localhost:3000

Special considerations

Typescript

At Ramp, we use React + Typescript in our codebase.

You are not required to know Typescript and using it in this challenge is optional. We have abstracted most of the Typescript code into its own files (types.ts), so feel free to ignore those. All of the bugs can be solved without Typescript.

If you work on the CodeSandbox, you can ignore any warnings on the code as long it works in the browser. However, feel free to write any Typescript code if your feel comfortable.

If you work locally, TSC_COMPLE_ON_ERROR flag is set to true by default. However, if you feel comfortable with Typescript, feel free to remove it on .env and to write any Typescript code.

API

We don't have a real API for this challenge, so we added some utilities to simulate API requests. To help you debug, we console.log the status of the ongoing simulated requests. You will not be able to see these requests in the network tab of your browser.

Solution

  • Solutions can be HTML, CSS or Javascript oriented, depending on the bug and your solution.
  • Modify any file inside the src folder as long as the expected result is correct.
  • The goal is to solve the bug as expected. Finding a clean and efficient solution is a nice to have, but not required.
  • Except for the last one, the first bugs don't depend on each other and can be solved in any order.
    • We recommend reading all the descriptions first. You might find the solution to one bug while trying to fix another.
    • The last bug will need other bugs to be fixed first in order to be reproduced.
  • You cannot add any external dependency to the project. The bugs can be solved with vanilla HTML, CSS and Javascript.

Bug 1: Select dropdown doesn't scroll with rest of the page

How to reproduce:

  1. Make your viewport smaller in height. Small enough to have a scroll bar
  2. Click on the Filter by employee select to open the options dropdown
  3. Scroll down the page

Expected: Options dropdown moves with its parent input as you scroll the page

Actual: Options dropdown stays in the same position as you scroll the page, losing the reference to the select input

Solution Found that class selector "RampInputSelect--dropdown-container" had the property "position: fixed;". I commented out "position: fixed" and now the dropdown moves as expected.

Bug 2: Approve checkbox not working

How to reproduce:

  1. Click on the checkbox on the right of any transaction

Expected: Clicking the checkbox toggles its value

Actual: Nothing happens

Solution Determined that the input for the checkbox was working correctly, however the label made it impossible to click the checkbox. This was found by commenting out the "display: none" CSS tag in "index.css" for "RampInputCheckbox--input". The solution was to include "htmlFor={inputId}". Checkbox works as expected.

Bug 3: Cannot select All Employees after selecting an employee

How to reproduce:

  1. Click on the Filter by employee select to open the options dropdown
  2. Select an employee from the list
  3. Click on the Filter by employee select to open the options dropdown
  4. Select All Employees option

Expected: All transactions are loaded

Actual: The page crashes

Solution Within the onChange event, there was no logic to render all employees once 'All Employees' was selectd. I added an if statement to check if "newValue.id" was equal to "EMPTY_EMPLOYEE.id" and if it were true, then to loadAllTransactions.

Bug 4: Clicking on View More button not showing correct data

How to reproduce:

  1. Click on the View more button
  2. Wait until the new data loads

Expected: Initial transactions plus new transactions are shown on the page

Actual: New transactions replace initial transactions, losing initial transactions

Solution Within "requests.ts" file I modified "const end = start + TRANSACTIONS_PER_PAGE" to instead take in the page value and the current value of TRANSACTIONS_PER_PAGE and then add onto that value. This now adds more information instead of overwriting existing information.

Bug 5: Employees filter not available during loading more data

This bug has 2 wrong behaviors that will be fixed with the same solution

Part 1

How to reproduce:

  1. Open devtools to watch the simulated network requests in the console
  2. Refresh the page
  3. Quickly click on the Filter by employee select to open the options dropdown

Expected: The filter should stop showing "Loading employees.." as soon as the request for employees is succeeded

Actual: The filter stops showing "Loading employees.." until paginatedTransactions is succeeded

Part 2

How to reproduce:

  1. Open devtools to watch the simulated network requests in the console
  2. Click on View more button
  3. Quickly click on the Filter by employee select to open the options dropdown

Expected: The employees filter should not show "Loading employees..." after clicking View more, as employees are already loaded

Actual: The employees filter shows "Loading employees..." after clicking View more until new transactions are loaded.

Solution My initial assumptions were that the requests were waiting on a response before rendering however according to the console within the devTools it was showing that the array for "Employees" was sent and ready to use. By moving the "setIsLoading(false)" statement within "App.tsx" above the await statements, it allows for the fetched data to be used once the data is available. This however shows "no users" because there is no data until the employees array is available. Moving "setIsLoading(false)" after the "await employeeUtils.fetchAll()" allows for the list to be used once ready, and displays users properly.

Bug 6: View more button not working as expected

This bug has 2 wrong behaviors that can be fixed with the same solution. It's acceptable to fix with separate solutions as well.

Part 1

How to reproduce:

  1. Click on the Filter by employee select to open the options dropdown
  2. Select an employee from the list
  3. Wait until transactions load

Expected: The View more button is not be visible when transactions are filtered by user, because that is not a paginated request.

Actual: The View more button is visible even when transactions are filtered by employee. You can even click View more button and get an unexpected result

Part 2

How to reproduce:

  1. Click on View more button
  2. Wait until it loads more data
  3. Repeat these steps as many times as you can

Expected: When you reach the end of the data, the View More button disappears and you are not able to request more data.

Actual: When you reach the end of the data, the View More button is still showing and you are still able to click the button. If you click it, the page crashes.

Solution There are checks missing in the case of transactions loading, and when reaching the end of the provided data which results in the page crashing. Because the expected result is specific to users not having paginated requests, the "view more" button should not render. Furthermore, when clicking "view more", the page should not crash once you have reached the end of the data list. Adding in these checks to see if transactions AND paginatedTransactions AND paginatedTransactions.nextPage are not null then to render the button if this is true.

Bug 7: Approving a transaction won't persist the new value

You need to fix some of the previous bugs in order to reproduce

How to reproduce:

  1. Click on the Filter by employee select to open the options dropdown
  2. Select an employee from the list (E.g. James Smith)
  3. Toggle the first transaction (E.g. Uncheck Social Media Ads Inc)
  4. Click on the Filter by employee select to open the options dropdown
  5. Select All Employees option
  6. Verify values
  7. Click on the Filter by employee select to open the options dropdown
  8. Verify values

Expected: In steps 6 and 8, toggled transaction kept the same value it was given in step 2 (E.g. Social Media Ads Inc is unchecked)

Actual: In steps 6 and 8, toggled transaction lost the value given in step 2. (E.g. Social Media Ads Inc is checked again)

Solution After determining that the data that was being modified was not updating properly, I assumed that either the data was not being saved after the checkbox was clicked, or the data was being saved and not being fetched properly. Here I reviewed "./utils/fetch" and found that the "setTransactionApproval" should work which means that the data isn't saving in the cache once the transaction is approved. I then reviewed "TransactionPane.tsx" and included "clearCache()" to line 32 to first clear the cache of this instance and then "setApproved(newValue)" would then include the newly updated transaction.

Submission

IMPORTANT: Before sharing your CodeSandbox, open the email.txt file and replace your email on the only line of the file. Don't use any prefix or suffix, just your email.

You will submit a link to a CodeSandbox with your responses. Make sure your CodeSandbox is not Read only and can be edited, otherwise you will be disqualified. See Coding


Callouts

  • Don't remove existing data-testid tags. Otherwise, your results will be invalidated.
  • Other than the bugs, don't modify anything that will have a different outcome. Otherwise, your results might be invalidated.
  • Plagiarism is a serious offense and will result in disqualification from further consideration.

ramp's People

Contributors

p-ano-past avatar

Watchers

 avatar

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    ๐Ÿ–– Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. ๐Ÿ“Š๐Ÿ“ˆ๐ŸŽ‰

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google โค๏ธ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.