Skip to content

Update Report_ActionItem#9139

Merged
Justicea83 merged 9 commits into
mainfrom
luke-update-report-action-item
May 27, 2022
Merged

Update Report_ActionItem#9139
Justicea83 merged 9 commits into
mainfrom
luke-update-report-action-item

Conversation

@Luke9389

@Luke9389 Luke9389 commented May 23, 2022

Copy link
Copy Markdown
Contributor

Details

This PR adds the pending and error states to Report_ActionItem and uses them to set the opacity to 50%.

Fixed Issues

$ https://github.com/Expensify/Expensify/issues/211551

@Luke9389 Luke9389 requested a review from a team as a code owner May 23, 2022 22:45
@Luke9389 Luke9389 self-assigned this May 23, 2022
@Luke9389 Luke9389 requested a review from roryabraham May 23, 2022 22:45
@melvin-bot melvin-bot Bot requested review from Justicea83 and removed request for a team May 23, 2022 22:46
@Luke9389 Luke9389 changed the title Update Report_ActionItem [HOLD] Update Report_ActionItem May 23, 2022
@Luke9389

Copy link
Copy Markdown
Contributor Author

I'm putting this on HOLD while I verify what needs to come before it. I think there may need to be some API PRs that happen first (not 100% sure yet though).

@Luke9389 Luke9389 changed the title [HOLD] Update Report_ActionItem Update Report_ActionItem May 24, 2022
@Luke9389

Copy link
Copy Markdown
Contributor Author

OK, I verified that this one is safe to merge without the other changes happening. The unpassed props evaluate to false because of the || casting them to booleans. 👍

Comment thread src/pages/home/report/reportActionPropTypes.js Outdated
@Luke9389 Luke9389 requested a review from Justicea83 May 24, 2022 23:49
Comment thread src/pages/home/report/reportActionPropTypes.js Outdated
Comment thread src/styles/StyleUtils.js Outdated
@Luke9389 Luke9389 requested a review from roryabraham May 26, 2022 20:47
@Luke9389

Copy link
Copy Markdown
Contributor Author

Updated

@roryabraham

Copy link
Copy Markdown
Contributor

All yours @Justicea83

@Justicea83 Justicea83 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code LGTM!

@Justicea83 Justicea83 merged commit 3e52b9a into main May 27, 2022
@Justicea83 Justicea83 deleted the luke-update-report-action-item branch May 27, 2022 09:27
@OSBotify

Copy link
Copy Markdown
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to staging by @Justicea83 in version: 1.1.69-0 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by @luacmartins in version: 1.1.69-2 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

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.

4 participants