Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

[RKM66-24] [BE] Sync registration field #1

Merged
merged 7 commits into from
Jul 6, 2023
Merged

[RKM66-24] [BE] Sync registration field #1

merged 7 commits into from
Jul 6, 2023

Conversation

tonklaw
Copy link
Member

@tonklaw tonklaw commented Jul 6, 2023

LINEAR: RKM66-24

Change made

  • Bug fixes
  • New features
  • Breaking changes

Describe what you have done

  • Sync registration fields
  • Changed docker compose MySQL to Postgres

New Features

  • Added emerPhone
  • Added emerRelation
  • Added WantBottle (will fix to camel case)

Fix

  • Changed docker compose MySQL to Postgres

Others

  • Vaccination verification has not been removed

@tonklaw tonklaw self-assigned this Jul 6, 2023
@ImSoZRious
Copy link
Member

ImSoZRious commented Jul 6, 2023

I think it's my pipeline fault. Let me check it real quick

@ImSoZRious
Copy link
Member

Next time, create new branch, cherry-pick to dev and open pull request to beta.
Dev environment is to function like playground that you can push and deploy without approval. So, it can be pushed directly making it out-of-sync with beta very often (making it unable to become source branch for pull request).

@ImSoZRious
Copy link
Member

Also, create pull request on your new proto at https://github.com/isd-sgcu/rpkm66-proto

@ImSoZRious
Copy link
Member

actions/setup-go#328

@ImSoZRious
Copy link
Member

Sorry for the bad pipeline, I also never test on this job.

@ImSoZRious ImSoZRious merged commit 27f1edd into beta Jul 6, 2023
4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants