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

Fix storage sync checkbox initial state #305

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

Victor4X
Copy link
Contributor

Fixes #201
Fixes #303

As mentioned in #201 this was indeed the only required change.
Everything is now working as expected in my testing

@pointydev
Copy link

Would be great if this was pulled in as I just installed the extension and ran into this exact issue.

@maximedrouhin
Copy link

Hi @Gitoffthelawn! Since it seems you are now in charge of managing the project (#329 (comment)), would you mind accepting this minor PR? In addition to the author, I also confirmed locally that everything works.

@Gitoffthelawn
Copy link
Collaborator

@maximedrouhin Yes, sounds good. I will hopefully get to it very soon. I need to take a look at a few branching techniques on GitHub. Are you an expert in that area by any chance? It may be helpful to have someone to ask if I have questions.

@maximedrouhin
Copy link

maximedrouhin commented Jun 15, 2023

I'm far from being an expert, but I can still try to give you advice based on my limited knowledge.
Basically, to handle contributions and to keep things working well, I think these are the first three levels:


Level 1: You basically just review PRs, and maybe get a bit of assistance to do it.
You let volunteers suggest features and report bugs (via the Issues tab in GitHub), and you let volunteers improve the code base via Pull Requests. You can also participate in this.
Your role is to check Pull Requests:

  • You clone the repo that the volunteer wants to merge into this repo
  • You check if you can use the extension normally in general
  • You have a look to the code changes and check if you understand them and if everything seems good - directly in GitHub for example. You can use ChatGPT or similar to explain changes to you if you want.
  • You check if you can use the extension normally in the specific areas where code changes were made
    If everything looks fine, you can merge the PR.
    You can also identify people that you trust and let them do this for you after ensuring that they understood that they have to check the code changes and test the extension before merging.

Level 2: You add automated checks to ensure things always work well in the master branch.
You can automate the "check that the extension is working" part to a certain extent using Actions. I don't know that much about it, but apparently you can have automated checks that ensure that things work correctly, and block merging before the checks pass.
This can speed up reviewing PRs, and most importantly ensure that everything is always working well.


Level 3: More complex way to handle branches?
You could have a complex way to handle branches, but I think this is more useful for very big projects with many active contributors. In our case, things work fine by default: You have a master branch that is always up-to-date and where the extension works well, and volunteers work on a copy of the repo and do a PR when they think everything works well.


There is still the question of how to deploy the extension to Firefox and Chrome, and I don't know much about this. I just know you should use the "Tags" feature of Github to tag the current commit with a version number (https://github.com/einaregilsson/Redirector/tags) when you think the extension works perfectly and is ready for publishing.


It might be a good idea to create a Discord server for this project and link it in the README and in the description of the extension in both extension stores (Chrome and Firefox). Actually, this is the first thing I would do. This way you'll be able to get help easily, and people who are interested in helping this extension can chat about it. You can make something really basic, I'm pretty sure some people will want to be moderators and improve the channels. If you want, you can just create it and I'll make the channels. Or you can do it yourself, I think you can just copy most of the channels of this server and it should work pretty well.
Just in case, my Discord username is kodemax.

@maximedrouhin
Copy link

@Gitoffthelawn (just in case you didn't get notified)

@Gitoffthelawn
Copy link
Collaborator

@maximedrouhin Thank you for your helpful note Maxime. I just received it, due to more than expected going on here. I'm hoping to block out 2-3 hours of my time to focus and figure out a good plan. I'll likely have a couple specific questions on which you may be able to advise.

After some more administrative stuff, the next main course of action is to release a version with this patch and a couple other tweaks that should (hopefully) not result in any regressions.

@maximedrouhin
Copy link

@maximedrouhin Thank you for your helpful note Maxime. I just received it, due to more than expected going on here. I'm hoping to block out 2-3 hours of my time to focus and figure out a good plan. I'll likely have a couple specific questions on which you may be able to advise.

After some more administrative stuff, the next main course of action is to release a version with this patch and a couple other tweaks that should (hopefully) not result in any regressions.

@Gitoffthelawn Don't hesitate to ask questions to me, I'll try to answer as best I can! You can hit me up on Discord (username kodemax), or another app, just tell me which one and your username. I should be a lot more reactive this way 😅

Good luck with everything!

@Gitoffthelawn
Copy link
Collaborator

@maximedrouhin Thank you for the kind offer Maxime. I've made a note to contact you if I run into any snags or have any questions. Thanks again.

@piejanssens
Copy link

@Gitoffthelawn can we get this in, please? It's almost a year for a one-liner review :-)

@Gitoffthelawn Gitoffthelawn merged commit cb4d88b into einaregilsson:master Oct 25, 2024
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.

Storage Sync Check Box Having Opposite Effect Sync option unchecked even when sync is enabled
5 participants