Closed quinthar closed 2 months ago
Is this being worked on?
Still in review - pending a review from @roryabraham. Dropping my assignment since I'm not adding much value as a reviewer here
⚠️ Looks like this issue was linked to a Deploy Blocker here
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.
If a regression has occurred and you are the assigned CM follow the instructions here.
If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.
⚠️ Looks like this issue was linked to a Deploy Blocker here
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.
If a regression has occurred and you are the assigned CM follow the instructions here.
If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.
⚠️ Looks like this issue was linked to a Deploy Blocker here
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.
If a regression has occurred and you are the assigned CM follow the instructions here.
If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.
⚠️ Looks like this issue was linked to a Deploy Blocker here
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.
If a regression has occurred and you are the assigned CM follow the instructions here.
If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.
⚠️ Looks like this issue was linked to a Deploy Blocker here
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.
If a regression has occurred and you are the assigned CM follow the instructions here.
If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.
Reviewing
label has been removed, please complete the "BugZero Checklist".
The solution for this issue has been :rocket: deployed to production :rocket: in version 9.0.1-19 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:
If no regressions arise, payment will be issued on 2024-07-02. :confetti_ball:
For reference, here are some details about the assignees on this issue:
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
@c3024 bump on the checklist so we can pay you. Can you also link me your Upwork profile so I can send you a job?
@nkdengineer Upwork offer here: https://www.upwork.com/nx/wm/offer/102963265. Ping me on the issue when you've accepted.
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
bump on the checklist so we can pay you.
Done
Can you also link me your Upwork profile so I can send you a job?
Here it is - https://www.upwork.com/freelancers/~0105555e2f227dbf47
Can the pay be bumped up here? This is a large PR touching many files and there were some big revisions as well. Thanks!
The solution for this issue has been :rocket: deployed to production :rocket: in version 9.0.3-7 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:
If no regressions arise, payment will be issued on 2024-07-10. :confetti_ball:
For reference, here are some details about the assignees on this issue:
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
Upwork job price has been updated to $500
@nkdengineer please accept offer here: https://www.upwork.com/nx/wm/offer/103096930 @c3024 please accept offer here: https://www.upwork.com/nx/wm/offer/103096934
Both of you, please ping me on this issue when you've accepted.
@puneetlath thanks, accepted!
@c3024 has been paid. Just waiting for you now @nkdengineer.
@puneetlath Accepted thanks 🙇
All paid. Thanks everyone!
Problem:
Each comment in the server database is stored in
reportAction.message
in HTML form. However, for some reason lost to time, when we send that same comment out as an Onyx update, we:reportAction.originalMessage
in the update (which is confusing)message
object that contains a copy of thehtml
message, as well as a strippedtext
version.message
object inside a totally unnecessary one-element arrayThat means every comment is actually sent out three times (four, if you include the update to the
report
object, but that's out of scope for now). This is clearly wasteful in a number of ways:Additionally, it's particularly problematic when sending Onyx updates out via UrbanAirship, which is extremely limited in the payload sizes allowed -- with anything over a certain limit just dropped quietly and never delivered, causing "gaps" in our update stream (which require more network calls to "backfill" the Onyx data on app open, which makes things slow). Basically, this was just a mistake introduced for reasons we can't remember, but that we want to undo.
Solution:
To solve this, please:
originalMessage
, start using array-message
originalMessage.html
to usemessage.html || message[0].html
message.text
to dynamically strip the text frommessage.html || message[0].html
originalMessage
ormessage.text
or any other key within originalMessage:originalMessage
message
without an arrayThis issue is for phase 1 and phase 3.
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @puneetlath