akadmin/chatwoot
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

refactor: use UPSERT for consistency in document upload

Browse Source
This commit is contained in:
Tanmay Deep Sharma
2025-07-02 19:58:34 +05:30
parent 8f27ba081c
commit 235ce3479c
4 changed files with 16 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/javascript/dashboard/components-next/captain/pageComponents/document/DocumentForm.vue
+11 -9
View File
@@ -131,15 +131,16 @@ const handleSourceTypeChange = type => {
<div class="flex gap-2">
<button
type="button"
class="flex-1 p-3 rounded-lg border border-n-slate-6 hover:border-n-slate-7 transition-colors"
class="flex-1 p-3 rounded-lg bg-n-alpha-2 hover:border-n-slate-7 transition-colors focus:outline-none"
:class="{
'bg-primary-50 border-primary-500 text-primary-700':
'ring-2 ring-woot-500 text-woot-500 border-0':
state.sourceType === 'url',
'bg-n-alpha-2 text-n-slate-10': state.sourceType !== 'url',
'border-2 border-n-slate-7 text-n-slate-11':
state.sourceType !== 'url',
}"
@click="handleSourceTypeChange('url')"
>
<div class="flex flex-col items-center gap-2">
<div class="flex items-center justify-center gap-2">
<svg
width="20"
height="20"
@@ -157,15 +158,16 @@ const handleSourceTypeChange = type => {
</button>
<button
type="button"
class="flex-1 p-3 rounded-lg border border-n-slate-6 hover:border-n-slate-7 transition-colors"
class="flex-1 p-3 rounded-lg bg-n-alpha-2 hover:border-n-slate-7 transition-colors focus:outline-none"
:class="{
'bg-primary-50 border-primary-500 text-primary-700':
'ring-2 ring-woot-500 text-woot-500 border-0':
state.sourceType === 'pdf',
'bg-n-alpha-2 text-n-slate-10': state.sourceType !== 'pdf',
'border-2 border-n-slate-7 text-n-slate-11':
state.sourceType !== 'pdf',
}"
@click="handleSourceTypeChange('pdf')"
>
<div class="flex flex-col items-center gap-2">
<div class="flex items-center justify-center gap-2">
<svg
width="20"
height="20"
@@ -205,7 +207,7 @@ const handleSourceTypeChange = type => {
<div v-if="!state.selectedFile">
<FileUpload
accept="application/pdf"
:max-size-m-b="10"
:max-size-m-b="25"
placeholder="Select a PDF file"
upload-text="Click to select PDF or drag and drop"
drag-text="Drop PDF file here"
app/javascript/dashboard/components-next/fileUpload/FileUpload.vue
+1 -1
View File
@@ -183,7 +183,7 @@ const displayText = computed(() => {
}
.file-upload-zone.drag-over {
@apply border-primary-500 bg-primary-50/50;
@apply border-n-blue-9 bg-n-blue-2/50;
}
.file-upload-zone.disabled {
app/javascript/dashboard/routes/dashboard/captain/documents/Index.vue
+2 -2
View File
@@ -59,8 +59,8 @@ const handleCreateDialogClose = () => {
};
const handleCreateDialogSuccess = () => {
// Refresh the documents list after successful creation
fetchDocuments();
// Document is already added to the store optimistically, no need to refetch
// The store action 'uploadPdf' or 'create' already adds the document via commit(mutationTypes.ADD)
};
const handleAction = ({ action, id }) => {
app/javascript/dashboard/store/captain/document.js
+2 -1
View File
@@ -7,7 +7,8 @@ const actions = mutationTypes => ({
try {
const response = await CaptainDocumentAPI.uploadPdf(formData);
const { data } = response;
commit(mutationTypes.ADD, data.document);
// Use UPSERT like the create action for consistency
commit(mutationTypes.UPSERT, data);
commit(mutationTypes.SET_UI_FLAG, { creatingItem: false });
return Promise.resolve(data);
} catch (error) {