Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • Sign in / Register
Minds Backend - Engine
Minds Backend - Engine
  • Project overview
  • Repository
  • Issues 264
  • Merge Requests 27
  • CI / CD
  • Security & Compliance
  • Packages
  • Wiki
  • Snippets
  • Members
  • Collapse sidebar
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Minds
  • Minds Backend - EngineMinds Backend - Engine
  • Merge Requests
  • !435

Open
Opened 5 days ago by Emiliano Balbuena@edgebal
Report abuse

WIP: Feature flags from GitLab

  • Overview 4
  • Commits 1
  • Pipelines 1
  • Changes 10
0/3 threads resolved

Summary

Integrate GitLab onto feature flags management

Closes #966.

Steps to test

TBD

Estimated Regression Scope

Feature flags might not be correctly read or exported to frontend.

Deployment

New Settings

$CONFIG->set('unleash', [
    'apiUrl' => '', // URL to GitLab's Unleash server
    'instanceId' => '', // GitLab's Unleash instance ID
    'applicationName' => '', // GitLab Feature Flags environment
    'pollingIntervalSeconds' => 300, // Cache bust
    'metricsIntervalSeconds' => 15, // Metrics interval
]);
Edited 4 days ago by Emiliano Balbuena
Request to merge epic/feature-flags into master
The source branch is 6 commits behind the target branch
Open in Web IDE
Pipeline #107680862 failed for 104a2e41 on epic/feature-flags
          Requires 3 more approvals from Devs, Deployers, and QA.
          Rami Albatal
          Rami Albatal
          Martin Santangelo
          Martin Santangelo
          Guy Thouret
          Guy Thouret
          Olivia Madrid
          Olivia Madrid
          Marcelo Rivera
          Marcelo Rivera
          There are merge conflicts. Resolve these conflicts or ask someone with write access to this repository to merge it locally

          Closes #966

          Deletes source branch

          You can merge this merge request manually using the
          • Emiliano Balbuena @edgebal added Squad::Yellow scoped label 5 days ago

            added scoped label

          • Emiliano Balbuena @edgebal changed the description 5 days ago

            changed the description

          • Emiliano Balbuena @edgebal mentioned in issue #966 5 days ago

            mentioned in issue #966

          • Emiliano Balbuena @edgebal changed the description 4 days ago

            changed the description

          • Mark Harding
            Mark Harding @markeharding started a thread on the diff 1 hour ago
            Last updated by Mark Harding 1 hour ago
            Core/Features/Repository.php
            2 /**
            3 * Repository
            4 *
            5 * @author edgebal
            6 */
            7
            8 namespace Minds\Core\Features;
            9
            10 use Minds\Core\Config;
            11 use Minds\Core\Di\Di;
            12 use Minds\Core\Session;
            13 use Minds\UnleashClient\Config as UnleashConfig;
            14 use Minds\UnleashClient\Entities\Context;
            15 use Minds\UnleashClient\Unleash;
            16
            17 class Repository
            • Mark Harding
              Mark Harding @markeharding · 1 hour ago
              Owner

              This isn't following Repository interface, should it be called something else?

            • Mark Harding
              Mark Harding @markeharding · 1 hour ago
              Owner

              Perhaps Services/Unleash?

            • Please register or sign in to reply
          • Mark Harding
            Mark Harding @markeharding started a thread on the diff 1 hour ago
            Core/Config/Exported.php
            35 39 * @param BlockchainManager $blockchain
            40 * @param FeaturesManager $features
            36 41 */
            37 42 public function __construct(
            38 43 $config = null,
            39 44 $thirdPartyNetworks = null,
            40 45 $i18n = null,
            41 $blockchain = null
            46 $blockchain = null,
            47 $features = null
            42 48 ) {
            43 49 $this->config = $config ?: Di::_()->get('Config');
            44 50 $this->thirdPartyNetworks = $thirdPartyNetworks ?: Di::_()->get('ThirdPartyNetworks\Manager');
            45 51 $this->i18n = $i18n ?: Di::_()->get('I18n');
            46 52 $this->blockchain = $blockchain ?: Di::_()->get('Blockchain\Manager');
            53 $this->features = $features ?: Di::_()->get('Features');
            • Mark Harding
              Mark Harding @markeharding · 1 hour ago
              Owner

              Features\Manager to be consistent with other managers?

            • Please register or sign in to reply
          • Mark Harding
            Mark Harding @markeharding started a thread on the diff 1 hour ago
            composer.json
            53 56 {
            54 57 "type": "vcs",
            55 58 "url": "https://github.com/davegardnerisme/cruftflake.git"
            59 },
            60 {
            61 "type": "path",
            62 "url": "../../unleash-client-php"
            • Mark Harding
              Mark Harding @markeharding · 1 hour ago
              Owner

              Will this be git based/packagist when out of WIP?

            • Please register or sign in to reply
          Please register or sign in to reply
          Assignee
          Emiliano Balbuena's avatar
          Emiliano Balbuena @edgebal
          None
          Milestone
          None
          Time tracking
          No estimate or time spent
          1
          Labels
          Squad::Yellow
          Lock merge request
          Unlocked
          10
          10 participants
          user avatar
          Rami Albatal
          user avatar
          Martin Santangelo
          user avatar
          Guy Thouret
          user avatar
          Olivia Madrid
          user avatar
          Marcelo Rivera
          user avatar
          Brian Hatchet
          user avatar
          Mark Harding
          Reference: minds/engine!435

          Now you can access the merge request navigation tabs at the top, where they’re easier to find.

          More information and share feedback