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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add new links to Code Structure #2494

Merged

Conversation

gribnoysup
Copy link
Contributor

Add two new links to "Code Structure" FAQ section (#1785)

I have a question regarding labels for the links, sorry if this question sounds stupid 😸

I looked through the docs and it seems that link labels are always just article/issue name. But in case of issue #384, the actual ticket is more about a naming convention but the comment is about how code structure affects time travel in devtools. Should this be mentioned in link label somehow?

@gribnoysup gribnoysup mentioned this pull request Jul 9, 2017
@timdorr
Copy link
Member

timdorr commented Jul 9, 2017

You can make the link text whatever you want. If there's a more apt description, go ahead and change it to that.

@gribnoysup
Copy link
Contributor Author

@timdorr thank you for the quick response! I changed link names. Do you think these new ones are ok?

@timdorr
Copy link
Member

timdorr commented Jul 10, 2017

LGTM. Thanks!

@timdorr timdorr merged commit 7c68f34 into reduxjs:master Jul 10, 2017
@gribnoysup gribnoysup deleted the feature/add-links-to-code-structure branch July 10, 2017 16:19
@gribnoysup gribnoysup restored the feature/add-links-to-code-structure branch July 11, 2017 07:35
@gribnoysup gribnoysup deleted the feature/add-links-to-code-structure branch July 11, 2017 07:37
seantcoyote pushed a commit to seantcoyote/redux that referenced this pull request Jan 14, 2018
* Add new links to Code Structure

* Change link description

* Change discussion description
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.

2 participants