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 154
    • Issues 154
    • 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 #66824533 passed for 63b0b880 on brianhatchet:email_schema
      Requires approval from Devs. Approved by
      Emiliano Balbuena
      Emiliano Balbuena
      Martin Santangelo
      Martin Santangelo
      Rami Albatal
      Rami Albatal
      Marcelo Rivera
      Marcelo Rivera
      Ben Hayward
      Ben Hayward
      Mark Harding
      Mark Harding
      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 5
      • Pipelines 4
      • 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 3 hours ago

        approved this merge request

      • Emiliano Balbuena
        Emiliano Balbuena @edgebal started a discussion on an old version of the diff 2 hours ago
        Resolved by Brian Hatchet 2 hours ago
      • Emiliano Balbuena
        Emiliano Balbuena @edgebal started a discussion on the diff 2 hours ago
        Resolved by Brian Hatchet 2 hours ago
      • Emiliano Balbuena
        Emiliano Balbuena @edgebal started a discussion on an old version of the diff 2 hours ago
        Resolved by Brian Hatchet 1 hour ago
      • Emiliano Balbuena
        Emiliano Balbuena @edgebal started a discussion on the diff 2 hours ago
        Last updated by Brian Hatchet 1 hour 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 · 2 hours ago
          Developer

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

        • Brian Hatchet
          Brian Hatchet :speech_balloon: @brianhatchet · 1 hour ago
          Maintainer

          @markeharding ?

        Please register or sign in to reply
      • Emiliano Balbuena
        Emiliano Balbuena @edgebal started a discussion on an old version of the diff 2 hours ago
        Resolved by Brian Hatchet 1 hour ago
      • Emiliano Balbuena
        Emiliano Balbuena @edgebal started a discussion on an old version of the diff 2 hours ago
        Resolved by Brian Hatchet 1 hour ago
      • Brian Hatchet :speech_balloon: @brianhatchet added 36 commits 2 hours 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

      • Brian Hatchet :speech_balloon: @brianhatchet added 1 commit 1 hour ago

        added 1 commit

        • 63b0b880 - CR cleanup

        Compare with previous version

      • Emiliano Balbuena @edgebal approved this merge request 1 hour ago

        approved this merge request

      • 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
      Rami Albatal
      user avatar
      Marcelo Rivera
      user avatar
      Ben Hayward
      user avatar
      Mark Harding
      user avatar
      Brian Hatchet
      user avatar
      Emiliano Balbuena
      Reference: minds/engine!187