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

Fixes issue #419: Improve app by adding working params and fix the fi… #421

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

D-01576
Copy link
Member

@D-01576 D-01576 commented Sep 1, 2024

Fixes Issue

Closes #419

Changes Proposed

  • Updated the filter functionality to work with both GitHub and Eddie platforms.
  • Added Working params that will improve the app

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • The title of my pull request is a short description of the requested changes.
  • This PR does not contain plagiarized content.
  • All new and existing tests passed.

reviewers!!

Please review the updated filter functionality and let me know if any additional changes are needed.

…ms and fix the filter when switch to github or eddie
@D-01576
Copy link
Member Author

D-01576 commented Sep 1, 2024

Hey @Bashamega, please review this

@Bashamega
Copy link
Member

Why did you open a new pr?

@D-01576
Copy link
Member Author

D-01576 commented Sep 1, 2024

@Bashamega, I were unable to remove the package-lock.json file, that is why I opened this new PR but I also closed the previous PR

@Bashamega
Copy link
Member

@Bashamega, I were unable to remove the package-lock.json file, that is why I opened this new PR but I also closed the previous PR

No problem. The changes look good. Lets wait until @eddiejaoude merge it

@D-01576
Copy link
Member Author

D-01576 commented Sep 1, 2024

@Bashamega, Ok thanks

@D-01576
Copy link
Member Author

D-01576 commented Sep 1, 2024

@eddiejaoude, Please review this PR

@Bashamega
Copy link
Member

@eddiejaoude, Please review this PR

He might take time since he has a lot of notifications and work.

@D-01576
Copy link
Member Author

D-01576 commented Sep 6, 2024

@eddiejaoude, please review this PR

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[OTHER] Improve app by adding working params and fix the filter when switch to github or eddie
3 participants