Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create employee/hangle-rfid-event, workbench/hangle-barcode-event #47

Merged
merged 9 commits into from
May 7, 2024

Conversation

zemljanichka
Copy link
Contributor

No description provided.

Copy link
Contributor

@PaTara43 PaTara43 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я призываю @zemljanichka и @pp241103 потестить это. У Петра есть все поднятое и есть железо. У Зарины есть код. Карточку создам. Крутяк!

Как потестите - надо решить конфликт и я солью

Dockerfile Outdated
COPY --from=requirements-stage /tmp/version.txt /src/version.txt
HEALTHCHECK --interval=5s --timeout=3s --start-period=5s --retries=12 \
CMD curl --fail http://localhost:5000/docs || exit 1
ENTRYPOINT ["uvicorn", "app:app", "--host", "0.0.0.0", "--port", "5000"]
ENTRYPOINT ["uvicorn", "src.app:app", "--host", "0.0.0.0", "--port", "5000"]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Братская помощь:DD
@Berlimor а чего вдруг тут?)
Просто интересно. Там никакие другие импорты и вольюмы не поломаются?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я пытался запустить, но крашился вбд, ругался на это.

build:
context: ./
dockerfile: Dockerfile
volumes:
- "/etc/timezone:/etc/timezone:ro"
- "/etc/localtime:/etc/localtime:ro"
- "./unit-passports/:/src/unit-passports/"
- "./unit-certificates/:/src/unit-certificates/"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

богоугодно

@@ -17,7 +17,7 @@
from src.feecc_workbench.Messenger import MessageLevels, message_generator, messenger
from src.database.models import GenericResponse
from src.feecc_workbench.utils import check_service_connectivity
from src.feecc_workbench.WorkBench import WorkBench
from src.feecc_workbench.WorkBench import Workbench
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Наверное, тоже богоугодно

src/config.py Outdated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я смотрю, тут поработал black:D


logger.debug(f"Handling BARCODE event. String: {event.string}")

if WORKBENCH.state == State.PRODUCTION_STAGE_ONGOING_STATE:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@timofeev41, ты не помнишь, а зачем это было?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

вообще чето нет

@PaTara43
Copy link
Contributor

PaTara43 commented Apr 4, 2024

Ответьте на комменты и я солью. @timofeev41 смотрел все?

@zemljanichka zemljanichka merged commit b1976de into refactoring May 7, 2024
1 check failed
@PaTara43 PaTara43 deleted the ref/login branch June 5, 2024 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants