Merged
Conversation
I also converted the application from an API only to a regular rails app, assuming that a backend challenge would also be ok without the heavy frontend work needed to use an API.
The command was
rails new backend-challenge
That overrides the existing files and adds the missing ones to have a regular frontend + backend rails app.
Substitute default rails welcome page
We are already testing the behavior on the members controller spec but still...
Devise already tests the authenticate_member! method we are using so we don't need to test it again
It will handle the authentication The code is the result of running rails generate devise:install rails generate devise Member
Our .rspec file already requires the spec_helper on every spec file so move all the test config there to avoid having to require it on every file individually
It provides a classless styles, so good defaults with minimal changes to the markup
I went with sucker punch and not Sidekiq to avoid adding the broker dependency(again, Redis) and to allow easy testing for the reviewer
It will be used to store the member headings
We will be accessing urls so we need a way to record the results to have a completely offline test suite
This job will pull the headings for a member to be called after the member creation
We need to override the Devise controller to be able to access the newly created member
Part of the requirements, i used my own personal url shortner chiq.me
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requirements covered:
Missing requirements
I added this requirement in another PR:
Technical decisions
Reviewer Setup
Clone the candidate's branch
Execute the following:
Make a copy of the rubric and fill it in
Ensure that the requests specs are the same or improved
Add a short summary to the Rubric and share it in our engineering-coding-projects Slack channel