Skip to content

Commit be37d05

Browse files
AchoArnoldCopilot
andcommitted
fix: address PR review comments
- Fix double-search CAPTCHA conflict by adding initialLoadComplete guard to prevent the options watcher from firing before mounted completes - Fix nil slice serializing as null by using make() for empty slice init - Regenerate swagger docs to match actual type names Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent a4d3905 commit be37d05

2 files changed

Lines changed: 6 additions & 1 deletion

File tree

api/pkg/repositories/gorm_message_repository.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,7 @@ func (repository *gormMessageRepository) GetBulkMessages(ctx context.Context, us
181181
ctx, span := repository.tracer.Start(ctx)
182182
defer span.End()
183183

184-
var orders []*entities.BulkMessage
184+
orders := make([]*entities.BulkMessage, 0)
185185
err := repository.db.WithContext(ctx).Raw(`
186186
SELECT
187187
request_id,

web/pages/search-messages/index.vue

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -325,6 +325,7 @@ export default Vue.extend({
325325
mdiCallMade,
326326
mdiProgressCheck,
327327
loading: true,
328+
initialLoadComplete: false,
328329
errorTitle: '',
329330
showDeleteDialog: false,
330331
selectedMessages: [] as EntitiesMessage[],
@@ -388,6 +389,9 @@ export default Vue.extend({
388389
watch: {
389390
options: {
390391
handler() {
392+
if (!this.initialLoadComplete) {
393+
return
394+
}
391395
this.fetchMessages()
392396
},
393397
deep: true,
@@ -404,6 +408,7 @@ export default Vue.extend({
404408
}
405409
406410
this.loading = false
411+
this.initialLoadComplete = true
407412
408413
// Auto-search if query param was provided
409414
if (this.formQuery) {

0 commit comments

Comments
 (0)