pubjs_augmentation #4

Open

skeris wants to merge 4 commits from pubjs_augmentation into staging

pull from: pubjs_augmentation

merge into: SQuiz:staging

SQuiz:main

SQuiz:staging

SQuiz:fixLayout

Conversation 0 Commits 4 Files Changed 2 +378

skeris commented 2025-05-22 12:53:10 +00:00

Owner

Copy Link

та самая ночная задача, что я тебе пояснял. в чем суть проблемы: фронт сперва грузит сам скрипт, потом идёт за настройками скрипта. это получается долго, типа сперва один запрос, потом второй. иногда сеть глючит и может пол минуты идти settings и тому подобное. поэтому мне пришла в голову мысль - а зачем делать второй запрос, а то и третий и четвёртый, если вопросов много, если бэк может отдать их сразу при выдаче скрипта. поэтому надо в этом репозитории сделать ручки для этого. сперва я хотел отдельный сервис, но подумал потом, что практической выгоды от этого нет
в этой задаче есть 3 момента:

1. как обновлять скрипт. это довольно просто, просто когда фронт будет заливать этот скрипт, я из пайплайна стрельну PUT /pub.js . надо принять файл, подготовить его (по факту порвать на 2 подстроки, слева и справа от места, куда надо вставить данные) и забыть старый файл. вариант с двумя строками мне нравится потому их будут только читать, за редким исключением, значит мьютексом закрывать почти не нужно. т.е. всё таки нужно, но там swap операция получается, когда подгружают новый файл. т.е. чтение этих строк не нужно защищать до тех пор, пока не наступит момент замены строк. а если ещё и в хендлер передавать их не как поле, а как указатель, то необходимость в мьютексе вовсе отпадает
2. когда приходит запрос GET //pub.js надо получить все данные по этому опросу какие есть, как буд-то человек стрельнул /settings и в Writer записать сперва первую часть строки-шаблона, потом смаршаленные данные, потом вторую часть строки-шаблона
3. структуру данных нужно обсудить с Настей в общем чате.

пока никакого кеширования не ведём

та самая ночная задача, что я тебе пояснял. в чем суть проблемы: фронт сперва грузит сам скрипт, потом идёт за настройками скрипта. это получается долго, типа сперва один запрос, потом второй. иногда сеть глючит и может пол минуты идти settings и тому подобное. поэтому мне пришла в голову мысль - а зачем делать второй запрос, а то и третий и четвёртый, если вопросов много, если бэк может отдать их сразу при выдаче скрипта. поэтому надо в этом репозитории сделать ручки для этого. сперва я хотел отдельный сервис, но подумал потом, что практической выгоды от этого нет в этой задаче есть 3 момента: 1) как обновлять скрипт. это довольно просто, просто когда фронт будет заливать этот скрипт, я из пайплайна стрельну PUT /pub.js . надо принять файл, подготовить его (по факту порвать на 2 подстроки, слева и справа от места, куда надо вставить данные) и забыть старый файл. вариант с двумя строками мне нравится потому их будут только читать, за редким исключением, значит мьютексом закрывать почти не нужно. т.е. всё таки нужно, но там swap операция получается, когда подгружают новый файл. т.е. чтение этих строк не нужно защищать до тех пор, пока не наступит момент замены строк. а если ещё и в хендлер передавать их не как поле, а как указатель, то необходимость в мьютексе вовсе отпадает 2) когда приходит запрос GET /<qid>/pub.js надо получить все данные по этому опросу какие есть, как буд-то человек стрельнул /settings и в Writer записать сперва первую часть строки-шаблона, потом смаршаленные данные, потом вторую часть строки-шаблона 3) структуру данных нужно обсудить с Настей в общем чате. пока никакого кеширования не ведём

skeris added the

Kind/Feature

label 2025-05-22 12:53:10 +00:00

pasha1coil was assigned by skeris 2025-05-22 12:53:10 +00:00

skeris added 22 commits 2025-05-22 12:53:10 +00:00

Merge branch 'chTypes' into 'staging' ... d161ecac88

fix clickhousr types

See merge request backend/quiz/answerer!5

add redirect url to prod 2842b19df3

Update docker-compose.yaml 7c78439807

Update app.go f348eba848

Update app.go e2b4579972

Update app.go 0d872219c6

Update app.go 5e4052e53b

applied renamer bb2223dc21

main deployment rule f99f8e80c1

added logic with quiz status == ai f8adf9b338

added logic for send req to ai, neew work with client process and client response be718cfde3

added ai client for current version ai api 3d294ea1d6

- bba0c1a4f6

- aad18f161a

release the kraken ai c7d7f2fda8

ai create only text question f5fe8654df

add status to get settings for ai ac54bf45ee

fix 2e7d9fb5ee

added omitempty for settings firld in resp 17fdefdd83

update to mew imp common 4a2771541a

fix ai question len 524b1caabf

added field auditory in req settings 0c6e9a4b4c

pasha1coil added 1 commit 2025-05-22 15:50:28 +00:00

start create controllers and structs 988df9d658

pasha1coil added 1 commit 2025-05-27 09:26:24 +00:00

upd 3164b42bc4

pasha1coil added 1 commit 2025-05-27 13:25:47 +00:00

update frontend_model.go and GetScriptTemplate 1cfb0f1b86

pasha1coil added 1 commit 2025-05-29 14:57:44 +00:00

added normal json file format c901e52844

This pull request can be merged automatically.

This branch is out-of-date with the base branch

You are not authorized to merge this pull request.

View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin pubjs_augmentation:pubjs_augmentation

git checkout pubjs_augmentation

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

Compat/Breaking

Breaking change that won't be backward compatible

  

Kind/Bug

Something is not working

  

Kind/Documentation

Documentation changes

  

Kind/Enhancement

Improve existing functionality

  

Kind/Feature

New functionality

  

Kind/Security

This is security issue

  

Kind/Testing

Issue or pull request related to testing

  

Priority

Critical

The priority is critical

  

Priority

High

The priority is high

  

Priority

Low

The priority is low

  

Priority

Medium

The priority is medium

  

Reviewed

Confirmed

Issue has been confirmed

  

Reviewed

Duplicate

This issue or pull request already exists

  

Reviewed

Invalid

Invalid issue

  

Reviewed

Won't Fix

This issue won't be fixed

  

Status

Abandoned

Somebody has started to work on this but abandoned work

  

Status

Blocked

Something is blocking this issue or pull request

  

Status

Need More Info

Feedback is required to reproduce issue or to continue work

No Label

Compat/Breaking

Kind/Bug

Kind/Documentation

Kind/Enhancement

Kind/Feature

Kind/Security

Kind/Testing

Priority

Critical

Priority

High

Priority

Low

Priority

Medium

Reviewed

Confirmed

Reviewed

Duplicate

Reviewed

Invalid

Reviewed

Won't Fix

Status

Abandoned

Status

Blocked

Status

Need More Info

Milestone

Clear milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

No Assignees

pasha1coil

2 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: SQuiz/answerer#4

No description provided.