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.

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

Update _routes.yml #28395

Closed
wants to merge 1 commit into from
Closed

Update _routes.yml #28395

wants to merge 1 commit into from

Conversation

bfitzexpensify
Copy link
Contributor

@bfitzexpensify bfitzexpensify requested a review from a team as a code owner September 28, 2023 14:36
@bfitzexpensify bfitzexpensify self-assigned this Sep 28, 2023
@melvin-bot melvin-bot bot requested review from dangrous and removed request for a team September 28, 2023 14:37
@melvin-bot
Copy link

melvin-bot bot commented Sep 28, 2023

@dangrous Please 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]

@dangrous
Copy link
Contributor

Okay so I think (I'm not sure) that you don't have to do this part anymore. Do you mind checking in Slack to confirm? If so we should update those instructions. I might be working off of a faulty memory though!

- href: connect-ANZ
title: Connect ANZ Bank
icon: /assets/images/money-wings.svg
description: If you're an ANZ Visa customer, you can set up your cards to import transactions directly into Expensify and get a discount on your Expensify subscription.
Copy link
Member

Choose a reason for hiding this comment

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

Hi @bfitzexpensify, I'm assuming you're following steps from a SO. It might be outdated

We don't need to make any changes to _routes for adding new articles. It's auto generated by a script.
Please close the PR, thanks!

@rushatgabhane
Copy link
Member

@dangrous yeah you're right

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.

3 participants