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: add icons in timeline for windows #706

Merged
merged 2 commits into from
Nov 20, 2024

Conversation

tribhuwan-kumar
Copy link
Contributor

@tribhuwan-kumar tribhuwan-kumar commented Nov 20, 2024


name: add icons in timeline for windows
about: it can currently exe of installed apps using Get-AppxPackage for microsoft installed apps, from registry key for installed application from installer, and searches in potential path like start menu and and C:\windows to get the exe path by app name, then it extracts icons from .exe and .lnk (modified zmconnelly/windows-icons this crate to tribhuwan-kumar/windows-icons
title: #643
labels: ''
assignees: ''
/claim #643

ps: it took some time to load all icons, maybe some asynchronous issue, fixing it
Screenshot 2024-11-20 121950

fix: #643


description

related issue: #

type of change

  • bug fix
  • new feature
  • breaking change
  • documentation update

how to test

add a few steps to test the pr in the most time efficient way.

if relevant add screenshots or screen captures to prove that this PR works to save us time.

checklist

  • MOST IMPORTANT: this PR will require less than 30 min to review, merge, and release to production and not crash in the hand of thousands of users
  • i have read the CONTRIBUTING.md file
  • i have updated the documentation if necessary
  • my changes generate no new warnings
  • i have added tests that prove my fix is effective or that my feature works

additional notes

any other relevant information about the pr.

Copy link

vercel bot commented Nov 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
screenpipe ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 20, 2024 8:11am

@louis030195
Copy link
Collaborator

@tribhuwan-kumar nice!

just wondering - what happen if this fail for some reason, does it crash the timeline? should not

for example if cannot find powershell or something like this

otherwise looks good to me!

@tribhuwan-kumar
Copy link
Contributor Author

@tribhuwan-kumar nice!

just wondering - what happen if this fail for some reason, does it crash the timeline? should not

for example if cannot find powershell or something like this

otherwise looks good to me!

nope, it'll not crash the timeline
if it's can't find icon of any application it'll simply log error in console!

@louis030195
Copy link
Collaborator

/approve

Copy link

algora-pbc bot commented Nov 20, 2024

Could not find a claim that corresponds to the PR. Please add the /claim snippet to the PR body first.

@louis030195 louis030195 merged commit fa05865 into mediar-ai:main Nov 20, 2024
3 of 7 checks passed
@louis030195
Copy link
Collaborator

Could not find a claim that corresponds to the PR. Please add the /claim snippet to the PR body first.

@tribhuwan-kumar make sure to add claim

thanks for amazing work!

can you also do the linux one?

@tribhuwan-kumar
Copy link
Contributor Author

Could not find a claim that corresponds to the PR. Please add the /claim snippet to the PR body first.

@tribhuwan-kumar make sure to add claim

thanks for amazing work!

can you also do the linux one?

i'll add claim in linux pr :)

@tribhuwan-kumar
Copy link
Contributor Author

tribhuwan-kumar commented Nov 20, 2024

@louis030195
you opened new issue for linux one, probably i should claim it
/claim #643

@tribhuwan-kumar
Copy link
Contributor Author

Could not find a claim that corresponds to the PR. Please add the /claim snippet to the PR body first.

@tribhuwan-kumar make sure to add claim

thanks for amazing work!

can you also do the linux one?

@louis030195
i added the claim, maybe you've to approve again!

@louis030195
Copy link
Collaborator

/approve

Copy link

algora-pbc bot commented Nov 21, 2024

Could not find a claim that corresponds to the PR. Please add the /claim snippet to the PR body first.

@louis030195
Copy link
Collaborator

/tip $50 @tribhuwan-kumar

Copy link

algora-pbc bot commented Nov 21, 2024

🎉🎈 @tribhuwan-kumar has been awarded $50! 🎈🎊

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

Successfully merging this pull request may close these issues.

[bounty] impl icon fetch on windows (& linux)
2 participants