Skip to content

Next

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
    • Help
    • Submit feedback
    • Contribute to GitLab
    • Switch to GitLab Next
  • Sign in / Register
Minds Backend - Engine
Minds Backend - Engine
  • Project
    • Project
    • Details
    • Activity
    • Releases
    • Cycle Analytics
    • Insights
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
    • Locked Files
  • Issues 158
    • Issues 158
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 41
    • Merge Requests 41
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Registry
    • Registry
  • Packages
    • Packages
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Minds
  • Minds Backend - EngineMinds Backend - Engine
  • Merge Requests
  • !187

Open
Opened 2 weeks ago by Brian Hatchet@brianhatchet:speech_balloon:
  • Report abuse
Report abuse

email states, writing and reading from cassandra

Writing and reading email states from Cassandra

Refactored the user state emails into delegates\senders which are in charge of talking to all the managers and getting the data for the given user state

Once we get the other emails in line, we'll be able to query cassandra and just

Edited 1 week ago by Brian Hatchet

Check out, review, and merge locally

Step 1. Fetch and check out the branch for this merge request

git fetch https://gitlab.com/brianhatchet/engine.git email_schema
git checkout -b brianhatchet/engine-email_schema FETCH_HEAD

Step 2. Review the changes locally

Step 3. Merge the branch and fix any conflicts that come up

git fetch origin
git checkout origin/master
git merge --no-ff brianhatchet/engine-email_schema

Step 4. Push the result of the merge to GitLab

git push origin master

Note that pushing to GitLab requires write access to this repository.

Tip: You can also checkout merge requests locally by following these guidelines.

Request to merge brianhatchet:email_schema into master
Open in Web IDE
  • Email patches
  • Plain diff
Pipeline #66823454 passed for ec2a1c71 on brianhatchet:email_schema
      Requires 2 more approvals from Devs.
      Martin Santangelo
      Martin Santangelo
      Marcelo Rivera
      Marcelo Rivera
      Ben Hayward
      Ben Hayward
      Emiliano Balbuena
      Emiliano Balbuena
      Rami Albatal
      Rami Albatal
      Ready to be merged automatically. Ask someone with write access to this repository to merge this request

      Allows commits from members who can merge to the target branch

      Deletes source branch

      • Discussion 11
      • Commits 4
      • Pipelines 3
      • Changes 20
      5/6 discussions resolved
      • Loading...
      • Brian Hatchet :speech_balloon: @brianhatchet added 4 commits 1 week ago

        added 4 commits

        • 067bcab3...fbbe6842 - 2 commits from branch minds:master
        • 1cc12f34 - Merge remote-tracking branch 'upstream/master' into email_schema
        • 30a2f094 - Delegates for sending campaign emails

        Compare with previous version

      • Brian Hatchet :speech_balloon: @brianhatchet unmarked as a Work In Progress 1 week ago

        unmarked as a Work In Progress

      • Brian Hatchet :speech_balloon: @brianhatchet assigned to @brianhatchet 1 week ago

        assigned to @brianhatchet

      • Brian Hatchet :speech_balloon: @brianhatchet changed the description 2 times within 3 minutes 1 week ago

        changed the description 2 times within 3 minutes

      • Brian Hatchet :speech_balloon: @brianhatchet changed milestone to %sprint: Funny Frog 1 week ago

        changed milestone to %sprint: Funny Frog

      • Marcelo Rivera @eiennohi approved this merge request 1 hour ago

        approved this merge request

      • Emiliano Balbuena
        Emiliano Balbuena @edgebal started a discussion on the diff 43 minutes ago
        Resolved by Brian Hatchet 7 minutes ago
      • Emiliano Balbuena
        Emiliano Balbuena @edgebal started a discussion on the diff 43 minutes ago
        Resolved by Brian Hatchet 4 minutes ago
      • Emiliano Balbuena
        Emiliano Balbuena @edgebal started a discussion on the diff 43 minutes ago
        Resolved by Brian Hatchet 2 minutes ago
      • Emiliano Balbuena
        Emiliano Balbuena @edgebal started a discussion on the diff 43 minutes ago
        Last updated by Brian Hatchet 2 minutes ago
        Core/Email/Campaigns/EmailCampaign.php
        65 66
        66 67 return true;
        67 68 }
        69
        70 /**
        71 * Returns the short name of the class as the template name.
        72 */
        73 public function getEmailCampaignId()
        74 {
        75 return (new \ReflectionClass($this))->getShortName();
        • Emiliano Balbuena
          Emiliano Balbuena @edgebal · 43 minutes ago
          Developer

          I love this, but check with @markeharding if he's OK using Reflection on production.

        • Brian Hatchet
          Brian Hatchet :speech_balloon: @brianhatchet · 2 minutes ago
          Maintainer

          @markeharding ?

        Please register or sign in to reply
      • Emiliano Balbuena
        Emiliano Balbuena @edgebal started a discussion on the diff 43 minutes ago
        Resolved by Brian Hatchet 1 minute ago
      • Emiliano Balbuena
        Emiliano Balbuena @edgebal started a discussion on the diff 43 minutes ago
        Resolved by Brian Hatchet 1 minute ago
      • Brian Hatchet :speech_balloon: @brianhatchet added 36 commits 8 minutes ago

        added 36 commits

        • 30a2f094...38958c20 - 35 commits from branch minds:master
        • ec2a1c71 - Merge remote-tracking branch 'upstream/master' into email_schema

        Compare with previous version

      • You're only seeing other activity in the feed. To add a comment, switch to one of the following options.
      Please register or sign in to reply
      Assignee
      Brian Hatchet's avatar Brian Hatchet @brianhatchet
      Assign to
      sprint: Funny Frog
      Milestone
      sprint: Funny Frog
      Assign milestone
      None
      Time tracking
      No estimate or time spent
      0
      Labels
      None
      Assign labels
      • View project labels
      Lock merge request
      Unlocked
      7
      7 participants
      user avatar
      Martin Santangelo
      user avatar
      Marcelo Rivera
      user avatar
      Ben Hayward
      user avatar
      Emiliano Balbuena
      user avatar
      Rami Albatal
      user avatar
      Mark Harding
      user avatar
      Brian Hatchet
      Reference: minds/engine!187