refactor(reporting): remove console logs from member time sheets and reporting slice
This commit is contained in:
@@ -27,9 +27,7 @@ export const reportingTimesheetApiService = {
|
|||||||
|
|
||||||
getMemberTimeSheets: async (body = {}, archived = false): Promise<IServerResponse<IRPTTimeMember[]>> => {
|
getMemberTimeSheets: async (body = {}, archived = false): Promise<IServerResponse<IRPTTimeMember[]>> => {
|
||||||
const q = toQueryString({ archived });
|
const q = toQueryString({ archived });
|
||||||
console.log('getMemberTimeSheets body:', body);
|
|
||||||
const response = await apiClient.post(`${rootUrl}/time-reports/members/${q}`, body);
|
const response = await apiClient.post(`${rootUrl}/time-reports/members/${q}`, body);
|
||||||
console.log('getMemberTimeSheets response:', response);
|
|
||||||
return response.data;
|
return response.data;
|
||||||
},
|
},
|
||||||
|
|
||||||
|
|||||||
@@ -152,7 +152,6 @@ const timeReportsOverviewSlice = createSlice({
|
|||||||
setSelectOrDeselectProject: (state, action) => {
|
setSelectOrDeselectProject: (state, action) => {
|
||||||
const project = state.projects.find(project => project.id === action.payload.id);
|
const project = state.projects.find(project => project.id === action.payload.id);
|
||||||
if (project) {
|
if (project) {
|
||||||
console.log('setSelectOrDeselectProject', project, action.payload);
|
|
||||||
project.selected = action.payload.selected;
|
project.selected = action.payload.selected;
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
@@ -227,25 +226,22 @@ const timeReportsOverviewSlice = createSlice({
|
|||||||
state.loadingProjects = false;
|
state.loadingProjects = false;
|
||||||
});
|
});
|
||||||
builder.addCase(fetchReportingMembers.fulfilled, (state, action) => {
|
builder.addCase(fetchReportingMembers.fulfilled, (state, action) => {
|
||||||
console.log('fetchReportingMembers fulfilled', action.payload);
|
|
||||||
const members = action.payload.map((member: any) => ({
|
const members = action.payload.map((member: any) => ({
|
||||||
id: member.id,
|
id: member.id,
|
||||||
name: member.name,
|
name: member.name,
|
||||||
selected: true, // Default to selected
|
selected: true,
|
||||||
avatar_url: member.avatar_url, // Include avatar URL if needed
|
avatar_url: member.avatar_url,
|
||||||
email: member.email, // Include email if needed
|
email: member.email,
|
||||||
}));
|
}));
|
||||||
state.members = members;
|
state.members = members;
|
||||||
state.loadingMembers = false;
|
state.loadingMembers = false;
|
||||||
});
|
});
|
||||||
|
|
||||||
builder.addCase(fetchReportingMembers.pending, state => {
|
builder.addCase(fetchReportingMembers.pending, state => {
|
||||||
console.log('fetchReportingMembers pending');
|
|
||||||
state.loadingMembers = true;
|
state.loadingMembers = true;
|
||||||
});
|
});
|
||||||
|
|
||||||
builder.addCase(fetchReportingMembers.rejected, (state, action) => {
|
builder.addCase(fetchReportingMembers.rejected, (state, action) => {
|
||||||
console.log('fetchReportingMembers rejected', action.payload);
|
|
||||||
state.loadingMembers = false;
|
state.loadingMembers = false;
|
||||||
console.error('Error fetching members:', action.payload);
|
console.error('Error fetching members:', action.payload);
|
||||||
});
|
});
|
||||||
|
|||||||
@@ -23,13 +23,11 @@ const Members: React.FC = () => {
|
|||||||
|
|
||||||
// Handle checkbox change for individual members
|
// Handle checkbox change for individual members
|
||||||
const handleCheckboxChange = (id: string, checked: boolean) => {
|
const handleCheckboxChange = (id: string, checked: boolean) => {
|
||||||
console.log('Select Change:', id);
|
|
||||||
dispatch(setSelectOrDeselectMember({ id, selected: checked }));
|
dispatch(setSelectOrDeselectMember({ id, selected: checked }));
|
||||||
};
|
};
|
||||||
|
|
||||||
// Handle "Select All" checkbox change
|
// Handle "Select All" checkbox change
|
||||||
const handleSelectAllChange = (e: CheckboxChangeEvent) => {
|
const handleSelectAllChange = (e: CheckboxChangeEvent) => {
|
||||||
console.log('Select All Change:', e);
|
|
||||||
const isChecked = e.target.checked;
|
const isChecked = e.target.checked;
|
||||||
setSelectAll(isChecked);
|
setSelectAll(isChecked);
|
||||||
dispatch(setSelectOrDeselectAllMembers(isChecked));
|
dispatch(setSelectOrDeselectAllMembers(isChecked));
|
||||||
|
|||||||
Reference in New Issue
Block a user