Skip to content

Commit 981b018

Browse files
authored
Docs/remove feedback (#3624)
* Remove Feedback page and redirect links to Getting Started * Remove legacy Gitbook docs/README page * Try simplifying Travis config to remove examples
1 parent 762cf67 commit 981b018

File tree

7 files changed

+47
-128
lines changed

7 files changed

+47
-128
lines changed

.travis.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,5 +2,4 @@ language: node_js
22
node_js: node
33
cache: npm
44
script:
5-
- npm run examples:lint
6-
- npm run examples:test
5+
- npm run prepare

docs/Feedback.md

Lines changed: 0 additions & 13 deletions
This file was deleted.

docs/README.md

Lines changed: 0 additions & 72 deletions
This file was deleted.

docs/introduction/GettingStarted.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -153,6 +153,8 @@ The **[#redux channel](https://discord.gg/0ZcbPKXt5bZ6au5t)** of the **[Reactifl
153153

154154
You can also ask questions on [Stack Overflow](https://stackoverflow.com) using the **[#redux tag](https://stackoverflow.com/questions/tagged/redux)**.
155155

156+
If you have a bug report or need to leave other feedback, [please file an issue on the Github repo](https://github.com/reduxjs/redux)
157+
156158
## Should You Use Redux?
157159

158160
Redux is a valuable tool for organizing your state, but you should also consider whether it's appropriate for your situation. **Don't use Redux just because someone said you should - take some time to understand the potential benefits and tradeoffs of using it**.

website/_redirects

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -140,3 +140,5 @@
140140
/api/index.html /api/api-reference
141141
/recipes/ServerRendering.html* /recipes/server-rendering:splat
142142
/advanced/usagewithtypescript* /recipes/usage-with-typescript:splat
143+
144+
/feedback /introduction/getting-started#help-and-discussion

website/core/Footer.js

Lines changed: 41 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -5,17 +5,17 @@
55
* LICENSE file in the root directory of this source tree.
66
*/
77

8-
const React = require("react");
8+
const React = require('react')
99

1010
class Footer extends React.Component {
1111
docUrl(doc, language) {
12-
const baseUrl = this.props.config.baseUrl;
13-
return `${baseUrl}${language ? `${language}/` : ""}${doc}`;
12+
const baseUrl = this.props.config.baseUrl
13+
return `${baseUrl}${language ? `${language}/` : ''}${doc}`
1414
}
1515

1616
pageUrl(doc, language) {
17-
const baseUrl = this.props.config.baseUrl;
18-
return baseUrl + (language ? `${language}/` : "") + doc;
17+
const baseUrl = this.props.config.baseUrl
18+
return baseUrl + (language ? `${language}/` : '') + doc
1919
}
2020

2121
render() {
@@ -34,18 +34,12 @@ class Footer extends React.Component {
3434
</a>
3535
<div>
3636
<h5>Docs</h5>
37-
<a href={this.docUrl("introduction/getting-started")}>
37+
<a href={this.docUrl('introduction/getting-started')}>
3838
Getting Started
3939
</a>
40-
<a href={this.docUrl("introduction/core-concepts")}>
41-
Core Concepts
42-
</a>
43-
<a href={this.docUrl("basics/basic-tutorial")}>
44-
Basics
45-
</a>
46-
<a href={this.docUrl("advanced/advanced-tutorial")}>
47-
Advanced
48-
</a>
40+
<a href={this.docUrl('faq')}>FAQ</a>
41+
<a href={this.docUrl('basics/basic-tutorial')}>Tutorial</a>
42+
<a href={this.docUrl('api/api-reference')}>API Reference</a>
4943
</div>
5044
<div>
5145
<h5>Community</h5>
@@ -56,8 +50,9 @@ class Footer extends React.Component {
5650
>
5751
Stack Overflow
5852
</a>
59-
<a href="https://discord.gg/0ZcbPKXt5bZ6au5t">
60-
Discord
53+
<a href="https://discord.gg/0ZcbPKXt5bZ6au5t">Discord</a>
54+
<a href="/introduction/getting-started#help-and-discussion">
55+
Feedback
6156
</a>
6257
</div>
6358
<div>
@@ -77,32 +72,38 @@ class Footer extends React.Component {
7772
</div>
7873
</section>
7974
<section className="copyright">
80-
{this.props.config.copyright}<br />
81-
Some icons copyright <a
75+
{this.props.config.copyright}
76+
<br />
77+
Some icons copyright{' '}
78+
<a
8279
href="https://fontawesome.com/license/free"
83-
style={{color : "white"}}
80+
style={{ color: 'white' }}
81+
>
82+
Font Awesome
83+
</a>{' '}
84+
and{' '}
85+
<a href="https://thenounproject.com" style={{ color: 'white' }}>
86+
Noun Project
87+
</a>{' '}
88+
(
89+
<a
90+
href="https://thenounproject.com/term/check/1870817/"
91+
style={{ color: 'white' }}
8492
>
85-
Font Awesome
86-
</a> and <a
87-
href="https://thenounproject.com"
88-
style={{color : "white"}}
89-
>
90-
Noun Project
91-
</a> (<a
92-
href="https://thenounproject.com/term/check/1870817/"
93-
style={{color : "white"}}
94-
>
95-
Hassan ali
96-
</a>, <a
97-
href="https://thenounproject.com/term/debugging/1978252/"
98-
style={{color : "white"}}
99-
>
100-
ProSymbols
101-
</a>)
102-
</section>
93+
Hassan ali
94+
</a>
95+
,{' '}
96+
<a
97+
href="https://thenounproject.com/term/debugging/1978252/"
98+
style={{ color: 'white' }}
99+
>
100+
ProSymbols
101+
</a>
102+
)
103+
</section>
103104
</footer>
104-
);
105+
)
105106
}
106107
}
107108

108-
module.exports = Footer;
109+
module.exports = Footer

website/sidebars.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@
7676
"faq/react-redux",
7777
"faq/miscellaneous"
7878
],
79-
"Other": ["glossary", "troubleshooting", "feedback"],
79+
"Other": ["glossary", "troubleshooting"],
8080
"API Reference": [
8181
"api/api-reference",
8282
"api/createstore",

0 commit comments

Comments
 (0)