Skip to content
This repository has been archived by the owner on Nov 5, 2024. It is now read-only.

Fix issue 2946 #2948

Merged
merged 2 commits into from
Feb 12, 2024
Merged

Fix issue 2946 #2948

merged 2 commits into from
Feb 12, 2024

Conversation

sandeepjak2007
Copy link
Contributor

@sandeepjak2007 sandeepjak2007 commented Feb 12, 2024 •
edited
Loading

Pull Request (PR) Checklist

Please check if your pull request fulfills the following requirements:

  • The PR is submitted to the main branch.
  • I've read the Contribution Guidelines and my PR doesn't break the rules.
  • I've read the Architecture Guidelines.
  • I confirm that I've run the code locally and everything works as expected.
  • 🎬 I've attached a screen recoding of the changes.

Tip: drag & drop the video to the PR description.

What's changed?

Previous
Screenshot 2024-02-11 at 2 23 55 PM

Now
Screenshot 2024-02-12 at 9 15 01 AM

Describe with a few bullets what's new:

  • Image load in read me changed to new images and URL changed to correct ones which does exists.

💡 Tip: Please, attach screenshots and screen recordings. It helps a lot!

Does this PR close any GitHub Issues?

Check Ivy Wallet Issues.

Troubleshooting CI failures ❌

GitHub Actions failing? Read our CI Troubleshooting guide.

@ILIYANGERMANOV ILIYANGERMANOV merged commit 53ac7d4 into Ivy-Apps:main Feb 12, 2024
8 checks passed
@sandeepjak2007 sandeepjak2007 deleted the fix-issue-2946 branch February 12, 2024 14:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ILIYANGERMANOV ILIYANGERMANOV ILIYANGERMANOV approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BUG] Read me file issue
2 participants
@sandeepjak2007 @ILIYANGERMANOV