New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A history of Artsy's web frontend #489

Merged
merged 9 commits into from Oct 5, 2018

Conversation

Projects
None yet
7 participants
@ashfurrow
Member

ashfurrow commented Sep 19, 2018

This post has been a long time coming! I'm trying to document how and why Artsy Engineering got to where it is, in terms of front-end web technologies. Basically, I'm interested in tracing artsy.net from rails new gravity -> Graph + Backbone&CoffeeScript -> Ezel (for our many Backbone apps) -> React. I want each transition to be described in terms of the business goals that drove the changes.

It's still a work-in-progress, but what I'm looking for is: what technical details in the post need more clarification around driving business needs?

@peril-staging

This comment has been minimized.

Show comment
Hide comment
@peril-staging

peril-staging bot Sep 19, 2018

Contributor

Danger run resulted in 1 warning and 2 markdowns; to find out more, see the checks page.

Generated by 🚫 dangerJS

Contributor

peril-staging bot commented Sep 19, 2018

Danger run resulted in 1 warning and 2 markdowns; to find out more, see the checks page.

Generated by 🚫 dangerJS

ashfurrow added some commits Sep 26, 2018

@ashfurrow ashfurrow changed the title from WIP: A history of Artsy's web frontend to A history of Artsy's web frontend Sep 26, 2018

@ashfurrow

This comment has been minimized.

Show comment
Hide comment
@ashfurrow

ashfurrow Sep 28, 2018

Member

Cool, thanks for the feedback @sweir27! Really valuable stuff. I've made some changes in this commit:
26499ed Let me know what you think.

Probably won't get around to publishing this today, so let's aim for the beginning of next week?

Member

ashfurrow commented Sep 28, 2018

Cool, thanks for the feedback @sweir27! Really valuable stuff. I've made some changes in this commit:
26499ed Let me know what you think.

Probably won't get around to publishing this today, so let's aim for the beginning of next week?

@joeyAghion

One take-away from this is that front-end technology preferences change a lot in a short amount of time, even all the way down to the language (Ruby, Javascript, Typescript). I think that warrants a healthy amount of skepticism and humility when saying anything authoritatively. This write-up is very balanced in its assessments but still might benefit from discussing trade-offs or drawbacks of our choices.

ashfurrow added some commits Oct 3, 2018

@ashfurrow

This comment has been minimized.

Show comment
Hide comment
@ashfurrow

ashfurrow Oct 3, 2018

Member

@joeyAghion Thanks for the feedback – I've tried to expand a little bit more on the drawbacks and how we evaluate tech in this commit.

Member

ashfurrow commented Oct 3, 2018

@joeyAghion Thanks for the feedback – I've tried to expand a little bit more on the drawbacks and how we evaluate tech in this commit.

@ashfurrow

This comment has been minimized.

Show comment
Hide comment
@ashfurrow

ashfurrow Oct 5, 2018

Member

Okay! Definitely the longest-running draft blog post I've had for a while 😅 Thanks for all the feedback everyone, going to merge but will wait until next week to announce on Twitter.

Member

ashfurrow commented Oct 5, 2018

Okay! Definitely the longest-running draft blog post I've had for a while 😅 Thanks for all the feedback everyone, going to merge but will wait until next week to announce on Twitter.

@ashfurrow ashfurrow merged commit 6578195 into source Oct 5, 2018

3 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
deploy/netlify Deploy preview ready!
Details

@ashfurrow ashfurrow deleted the fe-history branch Oct 5, 2018

ashfurrow added a commit that referenced this pull request Oct 5, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment