Update dev-board.js #91
Conversation
Shuffle the order of devs This fix #90
|
Deploy request for reactjs-ph-website rejected. Rejected with commit f7172d3 https://docs.netlify.com/configure-builds/environment-variables/#sensitive-variable-policy |
There was a problem hiding this comment.
@xstable, hi! Awesome. Can you please extract this array function into src/utils.js. Thanks!
Another thing, we see that you're using this.
stackoverflow.com/questions/2450954/how-to-randomize-shuffle-a-javascript-array
This is fine. But can you use the shorter and more optimized version (Durstenfeld shuffle algorithm) found in that same SO question and update to use ES6 (change to arrow fn, let, const, etc). Thanks!
|
@codekcv anything fine with it now? |
All good, I'm closing this PR now since you opened a different one for this issue. Great work man! |
Description
Summary of Changes
Closes #90