Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[NEW] User Status Messages #12710

Closed
wants to merge 15 commits into from
Closed

[NEW] User Status Messages #12710

wants to merge 15 commits into from

Conversation

wreiske
Copy link
Contributor

@wreiske wreiske commented Nov 23, 2018

This pull request adds the ability for users to set a custom status message by editing their profile or using a new /status slash command.

Status messages show up in direct messages on the header, as well as in the user info sidebar.

UPDATE

This pull request has changed recently. Please see #12710 (comment)

@rodrigok
Copy link
Member

@engelgabriel @thiagosanchz what you guys think about this?

@geekgonecrazy
Copy link
Contributor

Nice! Would be great to figure out how to surface this where it could be accessed quicker. But I think this is great way to do it! Especially with the slash command which will make it instantly usable even on mobile

@iprok
Copy link

iprok commented Nov 28, 2018

Very usefull feature. Waiting for it in stable version.

@Hudell
Copy link
Contributor

Hudell commented Nov 29, 2018

I had been working on #9353 to try and get it merged for a while, maybe both PRs could be merged together as both seem to have features the other lack.

@richardt3
Copy link

Looks good - it's something my users keep asking for. Obviously would like to see it in the same place as the current status change, but the slash command makes up for it. Can we get this merged to live pls!

@geekgonecrazy
Copy link
Contributor

@Hudell how hard would this be? Are both using close enough approches one could be merged and then the other merged to extend the functionality?

@wreiske
Copy link
Contributor Author

wreiske commented Dec 7, 2018

I'm not sure. I haven't looked too far into the code for the other system. I think for the sake of getting this in, we might as well merge this (since it includes the slash command to allow mobile status changes). My team can commit to working on the extra elements needed to set status from within the UI, api rest, as well as working with the IOS and Android apps to get it implemented there.

@mickymiek
Copy link

mickymiek commented Jan 21, 2019

What's required to get this in? We're very interested in this feature. We could provide help too if needed :)

@engelgabriel @geekgonecrazy

@faziloub
Copy link

when will be merged guys ?? any plan ?

@wreiske
Copy link
Contributor Author

wreiske commented Mar 12, 2019

This will need to be re-worked since a lot of the surrounding code has been moved around.

I'm willing to put time towards this again, but only with @geekgonecrazy 's or others OK. I don't want to spend more time on a pull-request that is a no-go. Let me know!

@geekgonecrazy
Copy link
Contributor

geekgonecrazy commented Mar 12, 2019

I personally still like this and would like to have it in. Even if the UI changed a bit sometime after merged.

@engelgabriel @thiagosanchz what you guys think about this?

@engelgabriel @thiagosanchz Ping! Still would like thoughts :)

@sampaiodiego
Copy link
Member

a lot will need to be changed code wise though as I'm about to move everything away from meteor packages..

@wreiske
Copy link
Contributor Author

wreiske commented Mar 15, 2019

a lot will need to be changed code wise though as I'm about to move everything away from meteor packages..

Let me know when it's safe for me to start working on this again. I saw a commit recently that moved a bunch of stuff out of the meteor packages. I'll plan a weekend to hammer out the status messages if you think the current implementation will be okay as a starting point.

Essentially:

  • /status slash command to set status (requires no changes to mobile apps to immediately allow changing of statuses while they implement UI for it)
  • Change status on the accounts page
  • Display status in the user bar
  • Display status in the user info side panel

@Hudell
Copy link
Contributor

Hudell commented Mar 15, 2019

I'll be working on merging this (along with the other status PR too) next week.

@wreiske if you want to solve the conflicts on this one, it would be a huge help!

@wreiske
Copy link
Contributor Author

wreiske commented Mar 17, 2019

I'll be working on merging this (along with the other status PR too) next week.

@wreiske if you want to solve the conflicts on this one, it would be a huge help!

I've updated to the latest build... Also added an Edit Status modal dialog. Trying to figure out how to push/update this pull request. I may need to just start a new pull-request.

@wreiske wreiske reopened this Mar 17, 2019
@wreiske
Copy link
Contributor Author

wreiske commented Mar 17, 2019

Updated to latest greatest 1.0.0-develop.

Ready to merge!... please, before everything moves and I have to update it again... lol!

user-status-messages mp4

image

image

image

image

image

image

image

message-clip-preview mov

image

Admin Settings
image

@wreiske
Copy link
Contributor Author

wreiske commented Mar 17, 2019

Added rocket.chat output message when using the status slash command.

user-status-message-slash mp4

EDIT: Also added "too long"
image

Edit again: Added ability to edit user status from the admin UI.
image

@Hudell Hudell self-assigned this Mar 20, 2019
@@ -378,6 +380,7 @@ API.v1.addRoute('users.updateOwnBasicInfo', { authRequired: true }, {
email: this.bodyParams.data.email,
realname: this.bodyParams.data.name,
username: this.bodyParams.data.username,
statusMessage: this.bodyParams.data.statusMessage,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would be really cool if like setAvatar this was a rest call. Imagining the ability to set status from music players, and many other much more practical integrations.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Like my smart toaster! "toast is done".

I'll get this coded up when I wake up unless someone else wants to give it a whack before then.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, pushed. @geekgonecrazy please take a look at the changes and let me know if I should change it.

@rodrigok
Copy link
Member

@wreiske the tests are failing, can you check?

@wreiske
Copy link
Contributor Author

wreiske commented Mar 25, 2019

@wreiske the tests are failing, can you check?

Can do. Give me a few...

@wreiske
Copy link
Contributor Author

wreiske commented Mar 25, 2019

@wreiske the tests are failing, can you check?

@rodrigok fixed with 8ecbe83. There is something else failing for channels, but I don't think it's related to this.... I could be mistaken.

my meteor npm run chimp-test passes all the way until
image

Edit: Looks like the build is failing because https://github.com/lovell/sharp-libvips/releases/download/v8.7.0/libvips-8.7.0-linux-x64.tar.gz is returning a 502.

@geekgonecrazy
Copy link
Contributor

Edit: Looks like the build is failing because https://github.com/lovell/sharp-libvips/releases/download/v8.7.0/libvips-8.7.0-linux-x64.tar.gz is returning a 502.

looks to be working again. Must have been having an issue at github

@wreiske
Copy link
Contributor Author

wreiske commented Mar 25, 2019

Edit: Looks like the build is failing because https://github.com/lovell/sharp-libvips/releases/download/v8.7.0/libvips-8.7.0-linux-x64.tar.gz is returning a 502.

looks to be working again. Must have been having an issue at github

yeah. tests are passing now for mongo 4-0... looks like they are failing at creating a discussion. Unrelated to my changes i think?

@engelgabriel engelgabriel added this to the 1.1.0 milestone Apr 7, 2019
@rodrigok rodrigok modified the milestones: 1.1.0, 1.2.0 May 15, 2019
@wreiske
Copy link
Contributor Author

wreiske commented Jun 22, 2019

Closed in favor of #13933

@wreiske wreiske closed this Jun 22, 2019
@geekgonecrazy
Copy link
Contributor

🤗 thanks for helping with this feature!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants