Skip to content

ref: visitor endpoint, modified render-deploy.yaml#18

Merged
devcavin merged 1 commit into
mainfrom
server
Jul 2, 2026
Merged

ref: visitor endpoint, modified render-deploy.yaml#18
devcavin merged 1 commit into
mainfrom
server

Conversation

@devcavin

@devcavin devcavin commented Jul 2, 2026

Copy link
Copy Markdown
Owner

This pull request introduces several important changes to the visitor registration flow in the backend, primarily making the zoneId field mandatory for visitor registration and updating related service logic. Additionally, there is a minor update to the deployment workflow file.

Visitor Registration Changes:

  • Made the zoneId field in RegisterVisitorRequest a required (non-nullable) property, enforcing that all visitor registrations must specify a zone. (backend/src/main/kotlin/io/github/devcavin/backend/web/dto/visitor/VisitorDTOs.kt)
  • Updated VisitorService to remove the null check for zoneId, reflecting its new required status. (backend/src/main/kotlin/io/github/devcavin/backend/service/VisitorService.kt)

DevOps / Workflow:

  • Added a blank line to the top of .github/workflows/deploy-render.yaml for formatting consistency. (.github/workflows/deploy-render.yaml)

@devcavin devcavin merged commit e63c7e3 into main Jul 2, 2026
2 checks passed
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.

1 participant