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

Updates to address gnome extensions review #152

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

Conversation

toolstack
Copy link

@toolstack toolstack commented Feb 10, 2024

Should resolve #145

@AlexNabokikh
Copy link

@yilozt would you be so kind to review and approve this change?

@Hyperspeed1313
Copy link

Does this change not also need to be made to src/prefs.ts? I don't know much about extensions but it looks to be the same file in both locations. Is the version at src/prefs.ts for Gnome 44 and below?

@toolstack
Copy link
Author

Does this change not also need to be made to src/prefs.ts? I don't know much about extensions but it looks to be the same file in both locations. Is the version at src/prefs.ts for Gnome 44 and below?

This change is to src/prefs.ts so I'm not sure what your referring to here.

@Hyperspeed1313
Copy link

I'm not sure anymore either.

@toolstack
Copy link
Author

I don't think it matters anymore as it looks like the extension is dead 🤷

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.

Publish onto extensions.gnome.org?
3 participants