Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix for: IOU has pin option in LHN #20771

Merged
merged 12 commits into from
Jun 22, 2023
Merged

Fix for: IOU has pin option in LHN #20771

merged 12 commits into from
Jun 22, 2023

Commits on Jun 14, 2023

  1. Added reportID param for isMoneyRequestReport

    @Ollyws
    Ollyws committed Jun 14, 2023
    Configuration menu
    Copy the full SHA
    5b3f8e6 View commit details
    Browse the repository at this point in the history

  2. Check if is money request when showing pin option

    @Ollyws
    Ollyws committed Jun 14, 2023
    Configuration menu
    Copy the full SHA
    3643714 View commit details
    Browse the repository at this point in the history

  3. Don't show context menu if it's empty

    @Ollyws
    Ollyws committed Jun 14, 2023
    Configuration menu
    Copy the full SHA
    f2234a0 View commit details
    Browse the repository at this point in the history

  4. Run prettier

    @Ollyws
    Ollyws committed Jun 14, 2023
    Configuration menu
    Copy the full SHA
    9b12ff6 View commit details
    Browse the repository at this point in the history

  5. Move isMoneyRequestReport to end of line.

    @Ollyws
    Ollyws committed Jun 14, 2023
    Configuration menu
    Copy the full SHA
    11a0221 View commit details
    Browse the repository at this point in the history

Commits on Jun 15, 2023

  1. Added String type to JSDoc param 

    Co-authored-by: Rajat Parashar <parasharrajat@users.noreply.github.com>
    @Ollyws @parasharrajat
    Ollyws and parasharrajat authored Jun 15, 2023
    Configuration menu
    Copy the full SHA
    c7b61ca View commit details
    Browse the repository at this point in the history

  2. Check if is Object instead of using reportID param 

    Co-authored-by: Rajat Parashar <parasharrajat@users.noreply.github.com>
    @Ollyws @parasharrajat
    Ollyws and parasharrajat authored Jun 15, 2023
    Configuration menu
    Copy the full SHA
    775a739 View commit details
    Browse the repository at this point in the history

  3. Removed undefined report arg 

    Co-authored-by: Rajat Parashar <parasharrajat@users.noreply.github.com>
    @Ollyws @parasharrajat
    Ollyws and parasharrajat authored Jun 15, 2023
    Configuration menu
    Copy the full SHA
    2363632 View commit details
    Browse the repository at this point in the history

  4. Removed second undefined report arg 

    Co-authored-by: Rajat Parashar <parasharrajat@users.noreply.github.com>
    @Ollyws @parasharrajat
    Ollyws and parasharrajat authored Jun 15, 2023
    Configuration menu
    Copy the full SHA
    634fa14 View commit details
    Browse the repository at this point in the history

  5. Remove report JSDoc param and use reportOrID

    @Ollyws
    Ollyws committed Jun 15, 2023
    Configuration menu
    Copy the full SHA
    4e7eb43 View commit details
    Browse the repository at this point in the history

Commits on Jun 20, 2023

  1. Merge branch 'main' into issue-20255

    @Ollyws
    Ollyws committed Jun 20, 2023
    Configuration menu
    Copy the full SHA
    5bd209d View commit details
    Browse the repository at this point in the history

  2. Removed not showing popover for empty contextmenu

    @Ollyws
    Ollyws committed Jun 20, 2023
    Configuration menu
    Copy the full SHA
    5bc6630 View commit details
    Browse the repository at this point in the history