Skip to content

Commit

Permalink
Merge pull request #14 from Horizontal-org/hot-fix-explanations
Browse files Browse the repository at this point in the history
add explanation id on payload
  • Loading branch information
ValbuenaG committed Jun 17, 2023
2 parents 2f4fdf8 + 3360255 commit 4b6c048
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/components/Explanations/index.tsx
Expand Up @@ -98,7 +98,7 @@ export const Explanations: FunctionComponent<Props> = ({ initialData }) => {
text={e.text}
unselect={() => { changeSelected(null) }}
onUpdate={(text) => {
updateExplanation(e.index, text, e.position)
updateExplanation(e.index, text, e.position, e.id)
}}
/>
</ExplanationBox>
Expand Down
3 changes: 3 additions & 0 deletions src/fetch/question.ts
Expand Up @@ -10,6 +10,7 @@ interface SubmitPayload {
apps: App[],
}
explanations: {
id?: number;
position: string;
text: string;
}
Expand Down Expand Up @@ -101,7 +102,9 @@ export const useSubmit = () => {
fieldOfWork: selectedFieldsOfWork
},
explanations: explanations.map((e) => {
console.log(e)
return {
id: e.id,
position: e.position + '',
index: e.index + '',
text: e.text
Expand Down
7 changes: 4 additions & 3 deletions src/store/slices/explanation.ts
Expand Up @@ -4,6 +4,7 @@ export interface Explanation {
index: number;
text?: string;
position?: number;
id?: number;
}

export interface ExplanationsSlice {
Expand All @@ -13,7 +14,7 @@ export interface ExplanationsSlice {
explanationIndex: number;
addExplanation: (index: number, text?: string, position?: number) => void
setInitialExplanations: (explanations: Explanation[]) => void
updateExplanation: (index: number, text: string, position?: number) => void
updateExplanation: (index: number, text: string, position?: number, id?: number) => void
updateExplanations: (explanations: Explanation[]) => void
deleteExplanation: (index: number) => void
deleteExplanations: (componentId: number, componentType: string) => void
Expand Down Expand Up @@ -73,11 +74,11 @@ export const createExplanationsSlice: StateCreator<
explanations: state.explanations.filter(e => !toDelete.includes(+e.index))
}))
},
updateExplanation: (index, text, position) => {
updateExplanation: (index, text, position, id) => {
let oldExplanations = get().explanations.filter(e => e.index !== index)
const explanations = [
...oldExplanations,
{ index: index, text: text, position: position}
{ id, index: index, text: text, position: position}
].sort((a, b) => a.position - b.position)

set((state) => ({
Expand Down

0 comments on commit 4b6c048

Please sign in to comment.