Proof comments @mention not working consistently

raven
raven
edited 02/18/22 inSmartsheet Basics

Just started using proofs to gain stakeholder approval. Some of the approvers are shared to our sheet, many are not. In the comments on the proof, I am unable to @mentionallof the invited reviewers. I am only able to @mentionSOMEof them. We can't have stakeholders outside our team shared to the sheet as it contains confidential information. What am I missing in my understanding of proofs?

While I can't share screenshots as they'd all contain PII, here is a detailed summary of the proof:

  • Person who created the proof: shared to sheet as admin.
  • Proof current version is 2. Issue occurred on v. 1 and is still occurring on v.2. Reviewers that I cannot @mention were invited to review v.1 and v.2
  • Version 2:
    • Invited to review: 8 people - 2 shared to sheet as admins, 1 shared to sheet as editor can share, 5 not shared to sheet.
    • Approved: 3 people - 1 shared to sheet as admin, 1 shared to sheet as editor can share, 1 not shared to sheet. Can @mention all of them in a comment.
    • Requires changes: 0
    • Pending: 5 people - 1 shared to sheet as admin, 4 not shared to sheet. Can @mention the person shared to sheet, and only 2 of the 4 people that aren't shared.

To further complicate this, one of the people (not shared) that Icannot@mention, interacted with the proof version 1 and clicked "require changes" but never showed as such on v.1 (she is listed under pending). We know she clicked "require changes" because the proof owner got an email notification saying as such. Maybe the stakeholder unchecked it after? But still, why can't I @mention her or the other non-shared reviewer?

Answers

  • Genevieve P.
    Genevieve P. Employee Admin

    Hi@raven

    The @mention feature is specific to the underlying Smartsheet sheet, which means that you won't be able to @mention a user to send them a notification who isnotshared to the source sheet, even if they have been invited to view that specific Proof in the Proofing window.

    If you open the Proof directly from the row, then @mention someone who isn't shared to the sheet (by typing out their full email after the @ sign), you'll be prompted to give them permissions to see the sheet. (See:Collect feedback without approvals)

    Screen Shot 2022-02-24 at 3.00.27 PM.png

    If you're reviewing the proof from the separate ProofingWindow, you can click on the @ symbol to select a user either from the Sheet Sharing list or from the Invitation list to identify them in the comment.However, if the user you select is just a Reviewer and not shared to the sheet, they won't receive an email notification from this comment, since they're not shared to the underlying sheet.

    I know the Product team is always looking for feedback on how Proofing is being used and what enhancements would be helpful in the future; if you have a moment,please fill out this form hereto let them know about your use-case and request.

    Thanks!

    Genevieve

  • Genevieve P.
    Genevieve P. Employee Admin

    Hi@raven

    I've checked in with the Product team and I was incorrect about my statement above; youshouldbe able to @mention users from theexternalProofing Window using the @ symbol in the bottom right, including members you are invited as Proof Reviewers but are not shared to the sheet.

    In my own testing I was unable to send out email notifications this way (to users who were not shared to the sheet). Since I can replicate what you're describing, I've submitted a ticket on your behalf with our Support team to escalate this issue. They may reach out to you via email for further questions as we investigate the root cause. Thank you for bringing this to our attention!

    Cheers,

    Genevieve

  • @Genevieve P.thanks for the excellent follow up! I was thinking it would need a ticket submitted since I can @mentionsomeusers not shared to the sheet, but not all of them. Always good to discover these unexpected behaviors during a pilot/testing phase of a new feature :)

  • PDunn
    PDunn ✭✭

    @Genevieve Evanshas a resolution been provided regarding being able to @mention reviewers in the external proofing window that are not shared with the source sheet? This is a major issue as well for my team. We have a intake sheet that creative jobs are being managed via, but we don't want the requesters to have access to this. We invite them to review proofs via the external proofing window and other reviewers want to @mention those invited and discuss edits, but it doesn't work.

  • Genevieve P.
    Genevieve P. Employee Admin

    Hi@PDunn

    进一步的调查之后,团队证实了that this is actually expected behaviour, as outlined in my first response. There is no support for notifying external collaborators about mentions on the external page, as they're not shared to the sheet.

    我知道一个工作ddress notifications to non-shared collaborators will be looked at along with other improvements in the future. Please submit your feature request to the Product team by creating anIdea Postin theSmartsheet Product Feedback and Ideas topichere in the Community. This will allow other users to vote on your enhancement idea!

    Cheers,

    Genevieve

  • Neil Egsgard
    Neil Egsgard ✭✭✭✭

    @raven,@PDunn,@Genevieve P.,

    I did not see an idea request for the Proofing window @mention notification for users not shared to the sheet. I created an idea request.

    https://community.smartsheet.com/discussion/104950/proof-notifications-allow-mention-notifications-on-proofs-to-users-not-shared-to-sheet

  • Neil Egsgard
    Neil Egsgard ✭✭✭✭
    edited 05/08/23

    @raven,@PDunn,@Genevieve P.,

    I used WorkApps and was able to send a notification to someone who was not share to the entire sheet. I had to:

    1. create a WorkApps
    2. add the user to a role
    3. setup a report that only showed the records the user was approved to access
    4. Add the report to the role
    5. Mention the user on the proofing window via the report in WorkApps

    I have attached a screen shot of the notification.

Fixed, needed to add \"Value\" before the formula to remove the apostrophe.<\/p>


<\/p>

=VALUE(IFERROR(JOIN(DISTINCT(COLLECT([Customer PO Amount (USD)]@row:[Customer PO Amount (Local Currency)]@row, [Customer PO Amount (USD)]@row:[Customer PO Amount (Local Currency)]@row, <>\"\"))), \" \"))<\/p>"}]}},"status":{"statusID":3,"name":"Accepted","state":"closed","recordType":"discussion","recordSubType":"question"},"bookmarked":false,"unread":false,"category":{"categoryID":321,"name":"Smartsheet Basics","url":"https:\/\/community.smartsheet.com\/categories\/smartsheet-basics%2B","allowedDiscussionTypes":[]},"reactions":[{"tagID":3,"urlcode":"Promote","name":"Promote","class":"Positive","hasReacted":false,"reactionValue":5,"count":0},{"tagID":5,"urlcode":"Insightful","name":"Insightful","class":"Positive","hasReacted":false,"reactionValue":1,"count":0},{"tagID":11,"urlcode":"Up","name":"Vote Up","class":"Positive","hasReacted":false,"reactionValue":1,"count":0},{"tagID":13,"urlcode":"Awesome","name":"Awesome","class":"Positive","hasReacted":false,"reactionValue":1,"count":0}],"tags":[{"tagID":254,"urlcode":"formulas","name":"Formulas"}]},{"discussionID":109460,"type":"question","name":"IF \/ OR formula then Check a Box","excerpt":"I need a formula that IF a specific column has either 1 of 2 specific choices, then a BOX in another column is checked: =IF(([Study Status]@row = \"Active\/ Open to Accrual\", 1) OR([Study Status]@row = \"In Start-Up\", 1))","snippet":"I need a formula that IF a specific column has either 1 of 2 specific choices, then a BOX in another column is checked: =IF(([Study Status]@row = \"Active\/ Open to Accrual\", 1)…","categoryID":321,"dateInserted":"2023-08-25T13:30:39+00:00","dateUpdated":null,"dateLastComment":"2023-08-25T16:21:27+00:00","insertUserID":9250,"insertUser":{"userID":9250,"name":"Susan Swisher","url":"https:\/\/community.smartsheet.com\/profile\/Susan%20Swisher","photoUrl":"https:\/\/us.v-cdn.net\/6031209\/uploads\/defaultavatar\/nWRMFRX6I99I6.jpg","dateLastActive":"2023-08-25T16:23:49+00:00","banned":0,"punished":0,"private":false,"label":"✭✭✭✭✭"},"updateUserID":null,"lastUserID":161714,"lastUser":{"userID":161714,"name":"Carson Penticuff","url":"https:\/\/community.smartsheet.com\/profile\/Carson%20Penticuff","photoUrl":"https:\/\/us.v-cdn.net\/6031209\/uploads\/userpics\/B0Q390EZX8XK\/nBGT0U1689CN6.jpg","dateLastActive":"2023-08-26T01:04:51+00:00","banned":0,"punished":0,"private":false,"label":"✭✭✭✭✭✭"},"pinned":false,"pinLocation":null,"closed":false,"sink":false,"countComments":9,"countViews":45,"score":null,"hot":3385956126,"url":"https:\/\/community.smartsheet.com\/discussion\/109460\/if-or-formula-then-check-a-box","canonicalUrl":"https:\/\/community.smartsheet.com\/discussion\/109460\/if-or-formula-then-check-a-box","format":"Rich","tagIDs":[254],"lastPost":{"discussionID":109460,"commentID":392616,"name":"Re: IF \/ OR formula then Check a Box","url":"https:\/\/community.smartsheet.com\/discussion\/comment\/392616#Comment_392616","dateInserted":"2023-08-25T16:21:27+00:00","insertUserID":161714,"insertUser":{"userID":161714,"name":"Carson Penticuff","url":"https:\/\/community.smartsheet.com\/profile\/Carson%20Penticuff","photoUrl":"https:\/\/us.v-cdn.net\/6031209\/uploads\/userpics\/B0Q390EZX8XK\/nBGT0U1689CN6.jpg","dateLastActive":"2023-08-26T01:04:51+00:00","banned":0,"punished":0,"private":false,"label":"✭✭✭✭✭✭"}},"breadcrumbs":[{"name":"Home","url":"https:\/\/community.smartsheet.com\/"},{"name":"Get Help","url":"https:\/\/community.smartsheet.com\/categories\/get-help"},{"name":"Smartsheet Basics","url":"https:\/\/community.smartsheet.com\/categories\/smartsheet-basics%2B"}],"groupID":null,"statusID":3,"attributes":{"question":{"status":"accepted","dateAccepted":"2023-08-25T15:13:03+00:00","dateAnswered":"2023-08-25T14:14:39+00:00","acceptedAnswers":[{"commentID":392575,"body":"

Give this a try:<\/p>

=IF(OR([Study Status]@row = \"Active\/Open to Accrual\", [Study Status]@row = \"In Start-Up\"), 1)<\/p>"}]}},"status":{"statusID":3,"name":"Accepted","state":"closed","recordType":"discussion","recordSubType":"question"},"bookmarked":false,"unread":false,"category":{"categoryID":321,"name":"Smartsheet Basics","url":"https:\/\/community.smartsheet.com\/categories\/smartsheet-basics%2B","allowedDiscussionTypes":[]},"reactions":[{"tagID":3,"urlcode":"Promote","name":"Promote","class":"Positive","hasReacted":false,"reactionValue":5,"count":0},{"tagID":5,"urlcode":"Insightful","name":"Insightful","class":"Positive","hasReacted":false,"reactionValue":1,"count":0},{"tagID":11,"urlcode":"Up","name":"Vote Up","class":"Positive","hasReacted":false,"reactionValue":1,"count":0},{"tagID":13,"urlcode":"Awesome","name":"Awesome","class":"Positive","hasReacted":false,"reactionValue":1,"count":0}],"tags":[{"tagID":254,"urlcode":"formulas","name":"Formulas"}]},{"discussionID":109457,"type":"question","name":"Conditional Formatting (modified date not in the last 3 weeks)","excerpt":"I would like to use Conditional Formatting (highlight the entire row yellow) if the modified date is not within the last 3 weeks. So any row that has not been recently updated (last 3 weeks) should be highlighted in yellow. Is this spmething that can be done directly within conditional formatting or should I first create a…","snippet":"I would like to use Conditional Formatting (highlight the entire row yellow) if the modified date is not within the last 3 weeks. So any row that has not been recently updated…","categoryID":321,"dateInserted":"2023-08-25T12:33:14+00:00","dateUpdated":null,"dateLastComment":"2023-08-25T12:40:57+00:00","insertUserID":161267,"insertUser":{"userID":161267,"name":"Jef Snyders","title":"Jef Snyders","url":"https:\/\/community.smartsheet.com\/profile\/Jef%20Snyders","photoUrl":"https:\/\/us.v-cdn.net\/6031209\/uploads\/userpics\/4HJAEW33KBD0\/nXEKEZE5EQEV4.jpg","dateLastActive":"2023-08-25T13:32:31+00:00","banned":0,"punished":0,"private":false,"label":"✭✭"},"updateUserID":null,"lastUserID":161267,"lastUser":{"userID":161267,"name":"Jef Snyders","title":"Jef Snyders","url":"https:\/\/community.smartsheet.com\/profile\/Jef%20Snyders","photoUrl":"https:\/\/us.v-cdn.net\/6031209\/uploads\/userpics\/4HJAEW33KBD0\/nXEKEZE5EQEV4.jpg","dateLastActive":"2023-08-25T13:32:31+00:00","banned":0,"punished":0,"private":false,"label":"✭✭"},"pinned":false,"pinLocation":null,"closed":false,"sink":false,"countComments":2,"countViews":26,"score":null,"hot":3385935251,"url":"https:\/\/community.smartsheet.com\/discussion\/109457\/conditional-formatting-modified-date-not-in-the-last-3-weeks","canonicalUrl":"https:\/\/community.smartsheet.com\/discussion\/109457\/conditional-formatting-modified-date-not-in-the-last-3-weeks","format":"Rich","tagIDs":[437],"lastPost":{"discussionID":109457,"commentID":392553,"name":"Re: Conditional Formatting (modified date not in the last 3 weeks)","url":"https:\/\/community.smartsheet.com\/discussion\/comment\/392553#Comment_392553","dateInserted":"2023-08-25T12:40:57+00:00","insertUserID":161267,"insertUser":{"userID":161267,"name":"Jef Snyders","title":"Jef Snyders","url":"https:\/\/community.smartsheet.com\/profile\/Jef%20Snyders","photoUrl":"https:\/\/us.v-cdn.net\/6031209\/uploads\/userpics\/4HJAEW33KBD0\/nXEKEZE5EQEV4.jpg","dateLastActive":"2023-08-25T13:32:31+00:00","banned":0,"punished":0,"private":false,"label":"✭✭"}},"breadcrumbs":[{"name":"Home","url":"https:\/\/community.smartsheet.com\/"},{"name":"Get Help","url":"https:\/\/community.smartsheet.com\/categories\/get-help"},{"name":"Smartsheet Basics","url":"https:\/\/community.smartsheet.com\/categories\/smartsheet-basics%2B"}],"groupID":null,"statusID":3,"attributes":{"question":{"status":"accepted","dateAccepted":"2023-08-25T12:40:09+00:00","dateAnswered":"2023-08-25T12:37:47+00:00","acceptedAnswers":[{"commentID":392551,"body":"

There is no direct way to do this. Adding a checkbox helper column with something similar to this and then formatting based on the checkbox is the simplest way. You can even hide the extra column, as there is no reason for it to be visible.<\/p>

=IF([Modified Date]@row < TODAY(-21), 1, 0)<\/p>"}]}},"status":{"statusID":3,"name":"Accepted","state":"closed","recordType":"discussion","recordSubType":"question"},"bookmarked":false,"unread":false,"category":{"categoryID":321,"name":"Smartsheet Basics","url":"https:\/\/community.smartsheet.com\/categories\/smartsheet-basics%2B","allowedDiscussionTypes":[]},"reactions":[{"tagID":3,"urlcode":"Promote","name":"Promote","class":"Positive","hasReacted":false,"reactionValue":5,"count":0},{"tagID":5,"urlcode":"Insightful","name":"Insightful","class":"Positive","hasReacted":false,"reactionValue":1,"count":0},{"tagID":11,"urlcode":"Up","name":"Vote Up","class":"Positive","hasReacted":false,"reactionValue":1,"count":0},{"tagID":13,"urlcode":"Awesome","name":"Awesome","class":"Positive","hasReacted":false,"reactionValue":1,"count":0}],"tags":[{"tagID":437,"urlcode":"conditional-formatting","name":"Conditional Formatting"}]}],"initialPaging":{"nextURL":"https:\/\/community.smartsheet.com\/api\/v2\/discussions?page=2&categoryID=321&includeChildCategories=1&type%5B0%5D=Question&excludeHiddenCategories=1&sort=-hot&limit=3&expand%5B0%5D=all&expand%5B1%5D=-body&expand%5B2%5D=insertUser&expand%5B3%5D=lastUser&status=accepted","prevURL":null,"currentPage":1,"total":4991,"limit":3},"title":"Trending in Smartsheet Basics","subtitle":null,"description":null,"noCheckboxes":true,"containerOptions":[],"discussionOptions":[]}">

Trending in Smartsheet Basics