Closed LowProfileDog closed 8 years ago
I would like to attempt this.
@Ghalander Simon is already working on it, sorry! But check out the other issues. Most of the available ones are reserved for todayโs event here at Fullstack LA, but we will free them up tonight
๐๐ป๐ Hacktoberfest: Trick or Treat!
If you havenโt yet, sign up for Hacktoberfest to earn an exclusive T-Shirt. Plus Iโm sure we can teach you a cool trick or two in the process
๐ค What you will need to know
You should have worked with JavaScript, Node.js and Testing. This is a rather advanced issue, you will need to write tests yourself and then resolve the issue. If you havenโt done that before, better look for another issue :)
Some knowledge about Hapi will help but you can figure it out if you know the things above :)
๐ The Bug
A special aspect of Hoodie is that it works offline. A side effect of that is that you can use an application even without signing up. To see an example of that, open https://minutes.io/welcome and click on "Try it now".
What we want to do now is to save the timestamp the user opens the app for the first time, as well as the timestamp the user eventually signs up (creates account on the server using
hoodie.account.signUp()
Right now, neither of these timestamps are stored.
:dart: The goal
The
PUT /sessions/account
request thathoodie.account.signUp()
sends will include acreatedAt
property besides username, password and id. If itโs not, set it to the current time (new Date().toISOString()
).Additionally, add the property
signedUpAt
and set it to the current time.That way we can later do analysis on our appโs users in terms of how long it took them to sign up after they started using the app :)
:clipboard: Step by Step
If this is your first, welcome :tada: :smile: Here is a great tutorial on how to send a pull request using the terminal.
Please ๐ only claim if you want to start working on it during the event.
Once claimed we add you as contributor to this repository.
In a nutshell: be patient and actively kind with us ๐
up for grabs
label within progress
.test(routes): signup timestamps
and push it either to your fork or push your new branch. Your tests should fail nowcloses hoodiehq/camp#61
in the description.in progress
label withready
. Ask in comments for a review :)๐คโ Questions
Ping us in the Hoodie Chat or on Twitter