Skip to content

Commit cbfe525

Browse files
authored
Merge pull request #658 from lazycipher/bugs-fix
Force merge
2 parents 42a20d7 + d0b67e6 commit cbfe525

File tree

12 files changed

+63
-55
lines changed

12 files changed

+63
-55
lines changed

src/App.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,15 +4,17 @@ import "./App.css";
44
import { Provider } from "react-redux";
55
import store from "./store";
66
import jwt_decode from "jwt-decode";
7-
import { setAuthToken } from "./utils/setAuthToken";
7+
import { allowCredentialsInHeader } from "./utils/allowCredentialsInHeader";
88
import { setCurrentUser, logoutUser } from "./actions/authAction";
99
import "./css/main.scss";
1010
import ReactGA from "react-ga";
1111

12+
allowCredentialsInHeader()
13+
1214
function App() {
1315
useEffect(() => {
1416
ReactGA.initialize("UA-173245995-1");
15-
setAuthToken()
17+
1618
});
1719
return (
1820
<Provider store={store}>

src/actions/adminAction.js

Lines changed: 17 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ import axios from 'axios'
22
import { errorHandler } from '../utils/errorHandler'
33
import { setRequestStatus } from '../utils/setRequestStatus'
44
import { SET_ADMIN, GET_ADMIN } from './types'
5-
import { setAuthToken } from '../utils/setAuthToken'
65
import jwt_decode from 'jwt-decode';
76
import { setCurrentUser } from './authAction'
87
import { BASE_URL } from './baseApi'
@@ -27,31 +26,27 @@ export const createAdmin = (adminInfo) => async (dispatch) => {
2726
export const loginAdmin = (adminInfo, history) => async (dispatch) => {
2827
try {
2928
const res = await axios.post(`${BASE_URL}/auth/login/`, adminInfo)
30-
dispatch(setRequestStatus(false));
31-
if (res.status === 200) {
29+
dispatch(setRequestStatus(false));
30+
if (res.status === 200) {
31+
dispatch(setRequestStatus(true));
3232

33-
const token = res.data.token;
34-
dispatch(setRequestStatus(true));
33+
// update state with user
34+
localStorage.setItem('userId', res.data.user)
35+
dispatch(setCurrentUser(res.data.user._id));
3536

36-
localStorage.setItem("jwtToken", (token));
37-
setAuthToken(token);
37+
// update localStorage with admin status
38+
localStorage.setItem('username', `${res.data.user.name.firstName} ${res.data.user.name.lastName}`)
39+
localStorage.setItem('admin', res.data.user.isAdmin)
40+
localStorage.setItem('ticketModerator', res.data.user.isTicketsModerator)
41+
localStorage.setItem('orgId', res.data.user.orgId);
3842

39-
// update state with user
40-
const decodedData = await jwt_decode(token);
41-
localStorage.setItem('userId', decodedData._id)
42-
dispatch(setCurrentUser(decodedData));
43-
44-
// update localStorage with admin status
45-
localStorage.setItem('admin', true)
46-
47-
dispatch({
48-
type: SET_ADMIN,
49-
payload: true
50-
})
51-
52-
history.push("/dashboard");
43+
dispatch({
44+
type: SET_ADMIN,
45+
payload: res.data.user.isAdmin
46+
})
47+
history.push("/dashboard");
5348
}
5449
} catch (error) {
5550
dispatch(errorHandler(error))
5651
}
57-
}
52+
}

src/actions/authAction.js

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import { SET_CURRENT_USER, GET_USER_PROFILE, PASSWORD_SUCCESSFULLY_CHANGED, PASSWORD_CHANGE_REQUEST_SUCCESS, SET_ADMIN } from './types';
22
import axios from 'axios';
3-
import { setAuthToken } from '../utils/setAuthToken';
43
import { errorHandler } from '../utils/errorHandler';
54
import { setRequestStatus } from '../utils/setRequestStatus';
65
import { BASE_URL } from './baseApi';
@@ -146,8 +145,6 @@ export const logoutUser = () => async (dispatch) => {
146145
const orgId = localStorage.getItem('orgId');
147146
localStorage.clear()
148147
localStorage.setItem('orgId', orgId)
149-
// delete authorization from the header
150-
setAuthToken(false);
151148
// set user to {}
152149
setCurrentUser({});
153150
// move to home

src/css/components/_modals.scss

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,12 +35,10 @@
3535
padding-top: 5px;
3636
}
3737
.search_btn {
38-
width: 6vw;
39-
height: 5vh;
4038
background: rgb(250, 251, 252);
4139
border-radius: 100px;
4240
color: #1a73e8;
43-
padding: 0.3em;
41+
padding: 0 1rem;
4442
margin-left: 1em;
4543
}
4644
.edit__post__option {

src/user/dashboard/Community/components/Toggle/ToggleSwitch.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ function ToggleSwitch(props) {
2121

2222
let handleClose = () => {
2323
console.log('close clicked!')
24+
setCheck(!checked)
2425
setModalShow(false)
2526
}
2627

@@ -99,4 +100,4 @@ const mapStateToProps = (state) => ({
99100
error: state.error
100101
})
101102

102-
export default connect(mapStateToProps)(withRouter(ToggleSwitch));
103+
export default connect(mapStateToProps)(withRouter(ToggleSwitch));

src/user/dashboard/news-feed/popups/AddEventModal.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ const AddEventModal = (props) => {
103103
>
104104
<Form.Label className="modal__label">Date</Form.Label>
105105
<Form.Control
106-
type="text"
106+
type="date"
107107
placeholder="YYYY/MM/DD"
108108
onChange={onEventDate}
109109
/>
@@ -116,7 +116,7 @@ const AddEventModal = (props) => {
116116
>
117117
<Form.Label className="modal__label">Time</Form.Label>
118118
<Form.Control
119-
type="text"
119+
type="time"
120120
placeholder="10:00 AM"
121121
onChange={onEventTime}
122122
/>

src/user/organization/popups/Admins.js

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ class Admins extends Component {
5656

5757
mapHelper = (allAdmins) => {
5858
let adminInfo = []
59-
if (allAdmins.length > 0) {
59+
if (allAdmins && allAdmins.length > 0) {
6060
allAdmins.forEach((admin) => {
6161
adminInfo.push({ name: admin.name.firstName + ' ' + admin.name.lastName, desc: admin.info.about?.designation, _id: admin._id, isRemoved: admin?.isRemoved || false })
6262
})
@@ -100,6 +100,15 @@ class Admins extends Component {
100100
const { onHide, show } = this.props
101101
const { inviteLink, whoCanSendInvite } = this.state
102102
const adminList = [...this.state.admins]
103+
104+
const handleModalHide = () => {
105+
this.setState({
106+
query: ''
107+
}, () => {
108+
onHide()
109+
})
110+
}
111+
103112
let admins = adminList.map((item) => (
104113
<Row className="modal__member" id="p1" key={item._id}>
105114
<div className="member__image">
@@ -123,7 +132,7 @@ class Admins extends Component {
123132
));
124133
return (
125134
<Modal
126-
onHide={onHide}
135+
onHide={handleModalHide}
127136
show={show}
128137
size="md"
129138
aria-labelledby="contained-modal-title-vcenter"
@@ -213,4 +222,4 @@ const mapStateToProps = (state) => ({
213222
org: state.org
214223
})
215224

216-
export default connect(mapStateToProps, { removeAdmin, getMember, getInviteLink, getOrgProfile })(Admins)
225+
export default connect(mapStateToProps, { removeAdmin, getMember, getInviteLink, getOrgProfile })(Admins)

src/user/organization/popups/Members.js

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ class Members extends Component {
8383

8484
mapHelper = (allMembers) => {
8585
let membersInfo = []
86-
if(allMembers.length > 0) {
86+
if(allMembers && allMembers.length > 0) {
8787
allMembers.forEach((member) => {
8888
membersInfo.push({
8989
name: member.name.firstName + ' ' + member.name.lastName,
@@ -113,6 +113,15 @@ class Members extends Component {
113113
const { onHide, show } = this.props
114114
const { isAdmin, inviteLink, whoCanSendInvite } = this.state
115115
const membersList = [ ...this.state.members]
116+
117+
const handleModalHide = () => {
118+
this.setState({
119+
query: ''
120+
}, () => {
121+
onHide()
122+
})
123+
}
124+
116125
let members = membersList.map((item) => (
117126
<Row className="modal__member" id="p1" key={item._id}>
118127
<div className="member__image">
@@ -142,7 +151,7 @@ class Members extends Component {
142151
));
143152
return (
144153
<Modal
145-
onHide={onHide}
154+
onHide={handleModalHide}
146155
show={show}
147156
size="md"
148157
aria-labelledby="contained-modal-title-vcenter"
@@ -239,4 +248,4 @@ export default connect(mapStateToProps, {
239248
getMember,
240249
getInviteLink,
241250
getOrgProfile,
242-
})(Members);
251+
})(Members);

src/user/proposals/ProposalEditor/EditorContent/DropZone.js

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import React, { useMemo, useCallback, useState } from "react";
1+
import React, { useMemo, useCallback, useState, useEffect } from "react";
22
import { useDropzone } from "react-dropzone";
33
import { toast, ToastContainer } from "react-toastify";
44
import "react-toastify/dist/ReactToastify.css";
@@ -33,7 +33,11 @@ const rejectStyle = {
3333
};
3434

3535
function StyledDropzone(props) {
36-
const [proposalId, setProposalId] = useState(props.idContent);
36+
const [proposalId, setProposalId] = useState(null);
37+
38+
useEffect(() => {
39+
setProposalId(props.idContent)
40+
}, [props.idContent])
3741

3842
const onDrop = useCallback((acceptedFiles) => {
3943
let formData = new FormData();
@@ -53,7 +57,7 @@ function StyledDropzone(props) {
5357
.catch((err) => {
5458
console.log(err);
5559
});
56-
}, []);
60+
}, [proposalId]);
5761

5862
const {
5963
getRootProps,

src/user/proposals/ProposalEditor/EditorContent/EditorContent.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -268,7 +268,7 @@ class EditorContent extends Component {
268268
<div>
269269
{!this.state.newProposal ? (
270270
<StyledDropzone
271-
idContent={this.props.location.state.proposalId}
271+
idContent={this.state.proposalId}
272272
/>
273273
) : (
274274
<div></div>

0 commit comments

Comments
 (0)