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
    • Dependency List
    • Cycle Analytics
    • Insights
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
    • Locked Files
  • Issues 192
    • Issues 192
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 27
    • Merge Requests 27
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Packages
    • Packages
    • List
    • Container Registry
  • 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
  • !232

Merged
Opened 1 month ago by Ben Hayward@benhayward.ben
  • Report abuse
Report abuse

[Sprint/HiperHedgehog](bug): Fixed forgot password validation

Need to setup SMTP

Closes #560 (closed)

Edited 5 days ago by Ben Hayward
Request to merge forgot-password-validation-fix-560 into master
  • Email patches
  • Plain diff
Pipeline #73167173 passed for 438ac593 on forgot-password-validation-fix-560
        Merge request approved. Approved by
        Brian Hatchet
        Brian Hatchet
        Mark Harding
        Mark Harding
        Emiliano Balbuena
        Emiliano Balbuena
        Martin Santangelo
        Martin Santangelo
        Rami Albatal
        Rami Albatal
        Marcelo Rivera
        Marcelo Rivera

        Merged by Mark Harding 14 minutes ago

        The changes were merged into master with 83d701b4

        The source branch has been deleted

        Closed #560 (closed)

        Pipeline #74125545 waiting for manual action for 83d701b4 on master
                    • Discussion 4
                    • Commits 2
                    • Pipelines 3
                    • Changes 1
                    1/1 thread resolved
                    • Loading...
                    • Mark Harding @markeharding added MR::Awaiting Review scoped label 1 month ago

                      added MR::Awaiting Review scoped label

                    • Marcelo Rivera @eiennohi approved this merge request 4 weeks ago

                      approved this merge request

                    • Mark Harding
                      Mark Harding @markeharding started a thread on an old version of the diff 2 weeks ago
                      Resolved by Ben Hayward 2 weeks ago
                    • Mark Harding @markeharding added MR::Requires Changes scoped label and automatically removed MR::Awaiting Review label 2 weeks ago

                      added MR::Requires Changes scoped label and automatically removed MR::Awaiting Review label

                    • Ben Hayward @benhayward.ben added 1 commit 2 weeks ago

                      added 1 commit

                      • e358c0a3 - Updated to use validate_password and fixed identation

                      Compare with previous version

                    • Ben Hayward @benhayward.ben changed the description 2 weeks ago

                      changed the description

                    • Ben Hayward @benhayward.ben resolved all threads 2 weeks ago

                      resolved all threads

                    • Ben Hayward
                      Ben Hayward @benhayward.ben · 2 weeks ago
                      Developer

                      Getting a weird behavior where it redirects anyway locally so will need to wait to test when we have email set up on the pods

                    • Ben Hayward @benhayward.ben added MR::Awaiting Review scoped label and automatically removed MR::Requires Changes label 2 weeks ago

                      added MR::Awaiting Review scoped label and automatically removed MR::Requires Changes label

                    • Mark Harding @markeharding added Squad::Green scoped label 2 weeks ago

                      added Squad::Green scoped label

                    • Ben Hayward @benhayward.ben changed the description 1 week ago

                      changed the description

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

                      I can't really see what has changed here because of the change in spaces.

                    • Mark Harding @markeharding added MR::Requires Changes scoped label and automatically removed MR::Awaiting Review label 1 week ago

                      added MR::Requires Changes scoped label and automatically removed MR::Awaiting Review label

                    • Ben Hayward
                      Ben Hayward @benhayward.ben · 6 days ago
                      Developer

                      Ah that would be 96 Mark, want me to take the spacing changes out?

                    • Ben Hayward @benhayward.ben added 1 commit 5 days ago

                      added 1 commit

                      • 438ac593 - changes to meet feedback

                      Compare with previous version

                    • Ben Hayward @benhayward.ben added MR::Awaiting Review scoped label and automatically removed MR::Requires Changes label 5 days ago

                      added MR::Awaiting Review scoped label and automatically removed MR::Requires Changes label

                    • Ben Hayward @benhayward.ben changed the description 5 days ago

                      changed the description

                    • Mark Harding @markeharding approved this merge request 2 days ago

                      approved this merge request

                    • Brian Hatchet :speech_balloon: @brianhatchet approved this merge request 1 day ago

                      approved this merge request

                    • Mark Harding @markeharding merged 14 minutes ago

                      merged

                    • Mark Harding @markeharding mentioned in commit 83d701b4 14 minutes ago

                      mentioned in commit 83d701b4

                    • 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
                    0 Assignees
                    None
                    Assign to
                    None
                    Milestone
                    None
                    Assign milestone
                    None
                    Time tracking
                    No estimate or time spent
                    2
                    Labels
                    MR::Awaiting Review Squad::Green
                    Assign labels
                    • View project labels
                    Lock merge request
                    Unlocked
                    7
                    7 participants
                    user avatar
                    Emiliano Balbuena
                    user avatar
                    Martin Santangelo
                    user avatar
                    Rami Albatal
                    user avatar
                    Marcelo Rivera
                    user avatar
                    Mark Harding
                    user avatar
                    Brian Hatchet
                    user avatar
                    Ben Hayward
                    Reference: minds/engine!232

                    Revert this merge request

                    This will create a new commit in order to revert the existing changes.

                    Switch branch
                    Cancel
                    A new branch will be created in your fork and a new merge request will be started.

                    Cherry-pick this merge request

                    Switch branch
                    Cancel
                    A new branch will be created in your fork and a new merge request will be started.