Replaced asset icons with fa icons

divyanshkul commented 3 years ago

Description

Using FontAwesome icons now
Deleted asset image files

Related Issues

Fixes #57

Relevant Screenshots/Videos

Checklist

Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[x] I read the Contributor Guide.
[x] I updated/added relevant comments in code.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change

Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

divyanshkul commented 3 years ago

Hahahaha LGTM as clear as it gets (btw it was showing merge errors?)

adityathakurxd commented 3 years ago

hehe yes! Got it from the official Flutter repo. Yep fixed the isssue. It was due to the latest merge.

divyanshkul commented 3 years ago

Oh thats why Less than a minute gap between the two PRs xD

divyanshkul commented 3 years ago

Code broke because of the merge error I'll send in a PR fixing it in a minute @adityathakurxd

adityathakurxd commented 3 years ago

Hey yes! I was working on it locally. It is fixed. Could you try latest pull?

divyanshkul commented 3 years ago

Yes, works now gg

adityathakurxd / university