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

new default icons for selected default labels + filter counter for overview in drawer #182

Open
wants to merge 11 commits into
base: develop
Choose a base branch
from

Conversation

Rushour0
Copy link

@Rushour0 Rushour0 commented Mar 4, 2023

Added default label icons
Made a new enum type - DefaultLabels which maps to the respective png files
A new default label can be added in DefaultLabels, with its corresponding png file in assets/icons with the same name as the label

Example :
label: videos
filename: videos.png

Added a class DefaultPaths
The intelli-sense can make it faster to write these paths, than having to remember them.

Example:
normal string path: assets/icons/
with DefaultPaths: DefaultPaths.defaultLabelIcons

The variable can be autocompleted with intellisense and reduces the need to remember the path if the variables are well named

solution to issue #108
label-icon-filter-screenshot

New function addition in the torrent service to count the torrents by filter
All counts of the filters are now visible in the drawer
issue #109 solution

label-count-screenshot

buildwithmalik and others added 8 commits September 5, 2022 13:06
…lytics (CCExtractor#169)

* test: Added tests for api service

* feat: Added Firebase analytics and crashlytics

* feat: Added environment variables for sensitive info

* Create .env
* test: Added tests for api service

* feat: Added Firebase analytics and crashlytics

* feat: Added environment variables for sensitive info

* Create .env

* Delete .env

* release: v0.9.1 beta release
@Rushour0
Copy link
Author

Rushour0 commented Mar 4, 2023

@AbdulMalikDev added the PR here

@buildwithmalik
Copy link
Collaborator

@Rushour0 please resolve the conflicts.

@Rushour0
Copy link
Author

Rushour0 commented Mar 4, 2023

@Rushour0 please resolve the conflicts.

Done @AbdulMalikDev

@buildwithmalik
Copy link
Collaborator

@Rushour0 Please also make sure the PR is passing all checks.

@Rushour0
Copy link
Author

Rushour0 commented Mar 5, 2023

@Rushour0 Please also make sure the PR is passing all checks.

Alright my bad, I will check it again, it was working some time ago

@Rushour0
Copy link
Author

Rushour0 commented Mar 5, 2023

@AbdulMalikDev cleared up the flutter analyze error

@Rushour0
Copy link
Author

i checked it last, was working fine
image

@Rushour0
Copy link
Author

Rushour0 commented Jun 27, 2023

@AbdulMalikDev could you let me know the status of this feature addition

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.

2 participants