Remove the change code below/above line instructions for Redux challenge:Use const for Action Types
See original GitHub issueDescribe your problem and how to reproduce it:
Hi there!
This challenge asks users to declare two variables with const and then edit the authReducer()
function and action creators.
The //change code below/above line
instructions imply that your code should only be between those two lines.
However, in order to pass the challenge, you have to edit the code below the // Change code above this line
.
I think it would be best to remove those comments.
Thanks!
Add a Link to the page with the problem: https://www.freecodecamp.org/learn/front-end-libraries/redux/use-const-for-action-types
Tell us about your browser and operating system:
- Browser Name:
- Browser Version:
- Operating System:
If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):
Issue Analytics
- State:
- Created 3 years ago
- Comments:13 (12 by maintainers)
Top Results From Across the Web
Redux Fundamentals, Part 3: State, Actions, and Reducers
The official Redux Fundamentals tutorial: learn how reducers update state in response to actions.
Read more >Redux createStore() is deprecated - Cannot get state from ...
But when I remove getState() , the action works. STORE: import { configureStore } from '@reduxjs/toolkit' import thunk from ...
Read more >4.2.0.12 - Adding Redux to React: Part 2
That means the store constant is a Redux store that knows how to handle ... We will update our ReactDOM.render() line of code...
Read more >Different Ways to Dispatch Actions with Redux - Pluralsight
We can then call the wrapped actions from our props. Below is the code: 1// MessageSender.container.js ...
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Since this is a fairly simple fix, I have open this PR up to first timers only.
@GJ-16 We are not planning on adding any more comments to the current curriculum. We have a specific set of comments located in /curriculum/dictionaries/english/comments.js that we want to stick to at the current time.
We have three specific sets comments we use related to where to put code:
and
and
My recommendation is to leave the existing comments at the top and add
Change this line
on thecase 'LOGIN':
andcase 'LOGOUT':
lines.