Open garrettmknight opened 4 months ago
Triggered auto assignment to @miljakljajic (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.
Who can post is set to All members
When we create a new workspace, the announce room has writeCapability
as undefined
in optimistic data and then BE returns it as all
.
We should pass writeCapability
as CONST.REPORT.WRITE_CAPABILITIES.ADMINS
here to fix the offline case for announce room
And BE also need to return writeCapability
of announce room as admin
in CreateWorkspace
API
NA
Job added to Upwork: https://www.upwork.com/jobs/~01d20a5670a3bdd03e
Triggered auto assignment to Contributor-plus team member for initial proposal review - @eVoloshchak (External
)
The fix is pretty straightforward, @nkdengineer's proposal looks good to me! Changes to BE would also be needed
πππ C+ reviewed!
Triggered auto assignment to @lakchote, see https://stackoverflow.com/c/expensify/questions/7972 for more details.
@eVoloshchak, @lakchote, @miljakljajic Eep! 4 days overdue now. Issues have feelings too...
@nkdengineer's proposal LGTM.
I'll take care of the BE changes.
π£ @nkdengineer π An offer has been automatically sent to your Upwork account for the Contributor role π Thanks for contributing to the Expensify app!
Offer link Upwork job Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review π§βπ» Keep in mind: Code of Conduct | Contributing π
We have a draft PR here @lakchote please let me know when BE is done
@eVoloshchak, @lakchote, @miljakljajic, @nkdengineer Whoops! This issue is 2 days overdue. Let's get this updated quick!
@eVoloshchak @lakchote @miljakljajic @nkdengineer this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!
@eVoloshchak, @lakchote, @miljakljajic, @nkdengineer 6 days overdue. This is scarier than being forced to listen to Vogon poetry!
Lucien is on parental leave. @miljakljajic Let's find another internal engineer to sort out the BE changes.
Current assignee @miljakljajic is eligible for the Bug assigner, not assigning anyone new.
@eVoloshchak can you type this again, Lucien is on leave and we're trying to trigger a new internal engineer:
πππ C+ reviewed!
Following this one as it's a very big pain point for Recorded Future that their non-admins can post to the #announce room by default.
πππ C+ reviewed!
Triggered auto assignment to @danieldoglas, see https://stackoverflow.com/c/expensify/questions/7972 for more details.
@danieldoglas, @eVoloshchak, @miljakljajic, @nkdengineer Whoops! This issue is 2 days overdue. Let's get this updated quick!
@danieldoglas are you good to assist with the BE changes required here?
Yes, I will.
Backend PR created
This is WIP, we need to change some automated tests on web so we can deploy it.
@danieldoglas, @eVoloshchak, @miljakljajic, @nkdengineer Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!
work continues on this one
@danieldoglas @eVoloshchak @miljakljajic @nkdengineer this issue is now 4 weeks old, please consider:
Thanks!
Yep, still working, other things took priority for now.
@danieldoglas, @eVoloshchak, @miljakljajic, @nkdengineer Eep! 4 days overdue now. Issues have feelings too...
still WIP
@danieldoglas, @eVoloshchak, @miljakljajic, @nkdengineer Still overdue 6 days?! Let's take care of this!
Still need to work on the Web part of the changes, Auth is done.
Still working
Yep. Lower priority compared to fast APIs now
I'll be OOO during the next 2 weeks - I'll finish this one once I'm back.
back from OOO
π£ It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? πΈ
@danieldoglas friendly bump
Sorry, there are some other competing priorities which are taking some time before this.
All backend PRs were created. We need to wait for the reviews/deploys - probably done by the end of the week
Last backend PR was merged, should be deployed Monday and we can close this.
This is done in the backend. @nkdengineer all yours to finish the app pr
@danieldoglas thank you, I'm working on this PR now
@danieldoglas Currently, after creating a new workspace, I am unable to access the announce room.
https://github.com/user-attachments/assets/2f7cf737-e4a9-4144-9f2e-d680b5f96365
You need your workspace with at least 4 people to have an announce room. This was changed recently.
@eVoloshchak this PR is ready for review
Merged, awaiting delpoy - should be deployed to staging today.
seems like this PR went out over 7 days ago, so I've paid @nkdengineer and providing a payment summary below for @eVoloshchak
@eVoloshchak is owed 250 for their work reviewing this issue.
If there are any further actions to take, please assign another BZ team member - I am out on extended leave from the end of the day.
If you havenβt already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: v1.4.73-7 Reproducible in staging?: y Reproducible in production?: y If this was caught during regression testing, add the test name, ID and link from TestRail: Email or phone of affected tester (no customers): N/A Logs: https://stackoverflow.com/c/expensify/questions/4856 Expensify/Expensify Issue URL: N/A Issue reported by: @garrettmknight Slack conversation: https://expensify.slack.com/archives/C05RECHFBEW/p1715886822520169
Action Performed:
Expected Result:
Who can post
should be defaulted toAdmin only
as it was designed in this docActual Result:
Who can post
is set toAll members
Workaround:
You can set it to admins only.
Platforms:
All
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @danieldoglas