Skip to content

Conversation

@ZIJ
Copy link
Contributor

@ZIJ ZIJ commented Oct 4, 2024

No description provided.

@delve-auditor
Copy link

delve-auditor bot commented Oct 4, 2024

Security

  • No security or compliance vulnerabilities detected
  • Scanned 7 changed file(s)

Changes Detected

  • [Refactor] Remove IsAcknowledged field, use DriftStatus instead [controllers/ci_jobs.go]

    • Updated ProjectDriftStateMachineApply function
    • Changed condition from project.IsAcknowledged to project.DriftStatus == dbmodels.DriftStatusAcknowledgeDrift
    • Improves consistency in drift status handling
  • [Feature] Add new fields to OrgSetting model [model/org_settings.gen.go]

    • Added ExternalOrgID field
    • Changed ID field type from int64 to string with UUID default
    • Updated corresponding field definitions in models_generated/org_settings.gen.go
  • [Refactor] Remove IsAcknowledged field from Project model [model/projects.gen.go]

    • Removed IsAcknowledged boolean field
    • Updated corresponding field definitions in models_generated/projects.gen.go
  • [Feature] Add new fields to User model [model/users.gen.go]

    • Added StripeCustomerID and IsPaying fields
    • Updated corresponding field definitions in models_generated/users.gen.go
  • [Refactor] Remove README.md file [ee/drift/README.md]

    • Deleted file containing "Backend for DriftApp" content

@ZIJ ZIJ merged commit 9545695 into develop Oct 4, 2024
breardon2011 pushed a commit that referenced this pull request Sep 17, 2025
Remove IsAcknowledged, use status instead
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.

2 participants