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

feat: ignore price database #40

Merged
merged 2 commits into from
Nov 2, 2021
Merged

feat: ignore price database #40

merged 2 commits into from
Nov 2, 2021

Conversation

MounirDhahri
Copy link
Member

@MounirDhahri MounirDhahri commented Nov 1, 2021

I apologize for the big diff, I alphabetically sorted the paths to make it easier to add new ones in the future.

This PR resolves https://artsyproduct.atlassian.net/browse/CX-2113

Description:
The goal of this PR is to make sure that the price-database link does not open the app and instead opens the link in the browser

Note:
The next steps here would be updating artsy-eigen-web-association in artsy-wwwify and force

Comment on lines -40 to -41
"NOT *L2FydGljbGVz*",
"NOT *L2FydGljbGUv*",
Copy link

Choose a reason for hiding this comment

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

@MounirDhahri This PR removes these two entries completely. Is that intended?

Copy link
Member Author

Choose a reason for hiding this comment

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

I also noticed that. These two has been deleted after I ran yarn build

Copy link

@lordkiz lordkiz Nov 1, 2021

Choose a reason for hiding this comment

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

Ok> Thanks for clarifying.

Copy link

@lordkiz lordkiz left a comment

Choose a reason for hiding this comment

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

🚀 . Looks good. I think we can merge if the deleted ones are not a problem.

@artsyit
Copy link
Contributor

artsyit commented Nov 2, 2021

🚀 PR was released in v1.1.0 🚀

@artsyit artsyit added the released This issue/pull request has been released. label Nov 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants