Closed rnithinpaladin closed 1 month ago
The recent changes revolve around introducing an asset governance feature to PacBot. A flag (asset_governance_enabled
) was added across various scripts and configurations to control this feature. New environment variables and configuration updates ensure that both the UI and backend respect this feature flag. The database schema and settings were also adjusted to accommodate this feature.
Files/Paths | Change Summary |
---|---|
installer/files/scripts/build_pacbot.py |
Added asset_governance_enabled flag and updated UI configuration logic accordingly. |
installer/resources/pacbot_app/build_ui_and_api.py |
Introduced ENV_ASSET_GOVERNANCE_FEATURE_FLAG to provisioners dictionary. |
installer/resources/pacbot_app/files/DB.sql |
Added @ASSET_GOVERNANCE_FEATURE_FLAG and updated pac_config_properties table changes. |
installer/resources/pacbot_app/import_db.py |
Added ASSET_GOVERNANCE_FEATURE_FLAG to environment variables dictionary. |
installer/settings/default.local.py |
Introduced ASSET_GOVERNANCE_FEATURE_FLAG with default False . |
webapp/src/config/configurations.ts |
Added assetGovernanceFeatureFlag to CONFIGURATIONS object with a default value of false . |
sequenceDiagram
participant User
participant WebApp
participant API
participant BuildPacBot
participant DB
User ->> WebApp: Access PacBot UI
WebApp ->> API: Request Asset Governance Status
API ->> BuildPacBot: Check Asset Governance Flag
BuildPacBot ->> DB: Retrieve Asset Governance Flag
DB -->> BuildPacBot: Return Flag Status
BuildPacBot -->> API: Asset Governance Flag Status
API -->> WebApp: Asset Governance Flag Status
WebApp -->> User: Display UI Based on Flag
In the world of PacBot's might, Asset governance takes flight. Flags are set, and features gleam, Bringing order to the dream. Configs updated, scripts in line, PacBot governance, truly fine. 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
⚠️ DELETE statement found in the changes of SQL files. Requesting review from @santhosh and @arun.
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
No data about Coverage
No data about Duplication
Description
Asset governance feature flag
Please include a summary of the changes and the related issues. Please also include relevant motivation and context. List any dependencies that are required for this change.
Problem
Solution
Fixes # (issue if any)
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:
Other Information:
List any documentation updates that are needed for the Wiki
Summary by CodeRabbit
New Features
Settings
ASSET_GOVERNANCE_FEATURE_FLAG
with a default value ofFalse
for enhanced governance control.