Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhance Docs w/ link to reagent component blog #437

Merged
merged 1 commit into from Aug 13, 2019

Conversation

athomasoriginal
Copy link
Contributor

@athomasoriginal athomasoriginal commented Jul 29, 2019

what the reagent component is a blog post that gets into the nitty gritty details of what it means when the docs say "underneath they are all just React components".

If there is anything missing from a technical perspective, I am happy to correct 馃槃

`what the reagent component` is a blog post that details in a little more detail what it means when "underneath they are all just React components".
@Deraen Deraen merged commit 2f4f9ac into reagent-project:master Aug 13, 2019
@Deraen
Copy link
Member

Deraen commented Aug 13, 2019

Thanks! Very nice post.

For those who want to know why the Reagent team chose to modify the pseudoclassical instantiation pattern I do not really have an answer. At the end of the day, they do more or less the same things without any significant downsides.

Me neither :) But I wasn't even aware of the differences, so I guess this will be something I'll look into, especially if there is something which could improve the performance of Reagent.

Note, in general I'd prefer having the information here in the Reagent docs instead of blog posts. It will be easier to update local docs.

@athomasoriginal
Copy link
Contributor Author

Note, in general I'd prefer having the information here in the Reagent docs instead of blog posts

Noted 馃槃 I was thinking about posting this in the docs, but it is...substantial lol. Thanks for the merge 馃憤

@athomasoriginal athomasoriginal deleted the patch-1 branch August 23, 2019 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants