Skip to content

Next

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
    • Help
    • Submit feedback
    • Contribute to GitLab
    • Switch to GitLab Next
  • Sign in / Register
Minds Mobile
Minds Mobile
  • Project
    • Project
    • Details
    • Activity
    • Releases
    • Dependency List
    • Cycle Analytics
    • Insights
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
    • Locked Files
  • Issues 137
    • Issues 137
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 8
    • Merge Requests 8
  • Registry
    • Registry
  • Packages
    • Packages
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Commits
  • Issue Boards
  • Minds
  • Minds MobileMinds Mobile
  • Merge Requests
  • !234

Open
Opened 3 days ago by Martin Santangelo@msantang78
  • Report abuse
Report abuse

[Sprint/HipsterHedgehog] client metadata and code refactor

  • client metadata added to views, votes
  • methods moved from the list stores to the models
  • fixed activity editor error
  • eslint with flow support added and configured
  • added view reporting to discovery and groups feeds (api/v2/analytics/views/)
  • added a debounce of 300ms in views reporting
  • added error messages to activity actions
  • fixed discovery gif optimization (when the image moves out of the screen)

closes #994 closes #993 closes #1000 closes #1002 closes #1003 closes #999 closes #953 closes #952 closes #997

Edited 5 hours ago by Martin Santangelo

Check out, review, and merge locally

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

git fetch https://gitlab.com/msantang78/mobile-native.git sprint/HipsterHedgehog.feat.metadata.code.cleaning
git checkout -b msantang78/mobile-native-sprint/HipsterHedgehog.feat.metadata.code.cleaning 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 msantang78/mobile-native-sprint/HipsterHedgehog.feat.metadata.code.cleaning

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 msantang78:sprint/HipsterHedgehog.feat.metadata.code.cleaning into master
The source branch is 16 commits behind the target branch
Open in Web IDE
  • Email patches
  • Plain diff
Could not retrieve the pipeline status. For troubleshooting steps, read the documentation.
Merge request approved. Approved by
Brian Hatchet
Brian Hatchet
Ready to be merged automatically. Ask someone with write access to this repository to merge this request

Closes #1000, #1002, #1003, #952, #953, #993, #994, #997, and #999

  • Discussion 1
  • Commits 7
  • Pipelines 4
  • Changes 54
0/1 discussion resolved
  • Loading...
  • Martin Santangelo @msantang78 added 1 commit 2 days ago

    added 1 commit

    • 70fcf25a - wip: client metadata and code refactor

    Compare with previous version

  • Martin Santangelo @msantang78 added 6 commits 16 hours ago

    added 6 commits

    • 5ef2e8e2 - (feat) add error log and message to votes
    • 4b0b0b9b - (fix) activity editor keeps changes on cancel
    • b07978e3 - (feat) client metadata, logic moved to models, view beacon refactor
    • 7624bda6 - (feat) eslint with flow added
    • a864d13f - (feat) implement flow in CommentList
    • e27ac3d2 - (chore) fix spec tests

    Compare with previous version

    Toggle commit list
  • Martin Santangelo @msantang78 added 2 commits 6 hours ago

    added 2 commits

    • bde8f72b - (chore) fix spec tests
    • 4cd49fbb - (chore) remove console.log

    Compare with previous version

  • Martin Santangelo @msantang78 unmarked as a Work In Progress 5 hours ago

    unmarked as a Work In Progress

  • Martin Santangelo @msantang78 changed milestone to %sprint: Hipster Hedgehog 5 hours ago

    changed milestone to %sprint: Hipster Hedgehog

  • Martin Santangelo @msantang78 changed the description 2 times within 6 minutes 5 hours ago

    changed the description 2 times within 6 minutes

  • Brian Hatchet
    Brian Hatchet :speech_balloon: @brianhatchet started a discussion on the diff 13 minutes ago
    .editorconfig
    10 indent_size = 2
    • Brian Hatchet
      Brian Hatchet :speech_balloon: @brianhatchet · 13 minutes ago
      Maintainer

      Did you mean to commit these changes?

    Please register or sign in to reply
  • Brian Hatchet :speech_balloon: @brianhatchet approved this merge request 13 minutes 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
None
Assign to
sprint: Hipster Hedgehog
Milestone
sprint: Hipster Hedgehog
Assign milestone
None
Time tracking
No estimate or time spent
0
Labels
None
Assign labels
  • View project labels
Lock merge request
Unlocked
2
2 participants
user avatar
Brian Hatchet
user avatar
Martin Santangelo
Reference: minds/mobile-native!234