Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
4.03k stars 3.03k forks source link

Update Configure-Netsuite.md #56391

Closed muttmuure closed 2 days ago

muttmuure commented 1 week ago

Explanation of Change

Fixed Issues

$https://github.com/Expensify/Expensify/issues/463224 PROPOSAL:

Tests

Offline tests

QA Steps

// TODO: These must be filled out, or the issue title must include "[No QA]."

PR Author Checklist

Screenshots/Videos

Android: Native
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari
MacOS: Desktop
melvin-bot[bot] commented 1 week ago

Concierge reviewer checklist:

For more detailed instructions on completing this checklist, see How do I review a HelpDot PR as a Concierge Team member?

cc @strepanier03

melvin-bot[bot] commented 1 week ago

@Pujan92 @strepanier03 One of you needs to copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

OSBotify commented 1 week ago

A preview of your ExpensifyHelp changes have been deployed to https://c7a82086.helpdot.pages.dev ⚡️

strepanier03 commented 1 week ago

@muttmuure - I just reviewed the build and noticed some formatting in an FAQ was off. It looks like the bullet points and returns aren't working.

image

Then, in the last FAQ, I think the two bullet items would stand out better as bullet points; what do you think?

image
strepanier03 commented 4 days ago

@muttmuure happy to review this again if you make changes, just let me know!

Pujan92 commented 4 days ago

@strepanier03 Can you unassign me as I think C+ review is not needed here

muttmuure commented 2 days ago

Hm, it looks like changes have already been made to main.

muttmuure commented 2 days ago

I'm not sure how to fix that right now so I'm going to create a new PR