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

Pull Request for #10: Implement autofill program #11

Merged
merged 4 commits into from
Jul 15, 2022

Conversation

veli-m-andirin
Copy link
Collaborator

@rcalvo12, could you please review the changes here? I brought in our autofill program from Protests with some slight improvements and a help file.

@jmshapir, could you please share write permission with @rcalvo12?

(If you think we should ask someone else for review, that is fine with me, too.)

@jmshapir
Copy link
Contributor

@veli-m-andirin thanks! I added @ew487 with read permissions for now. (We can elevate if needed.)

Can you please assign @ew487 as reviewer when @ew487 accepts the invitation to the repository?

Thanks both!

@veli-m-andirin
Copy link
Collaborator Author

Thanks @jmshapir! @ew487, could you please review the changes here? Please see the opening comment, and let me know if you have any questions.

@rcalvo12
Copy link

@veli-m-andirin Did you still want my review on this?

@veli-m-andirin
Copy link
Collaborator Author

Thanks @rcalvo12! I think one pair of eyes would suffice.

@veli-m-andirin veli-m-andirin merged commit 91d2c3b into master Jul 15, 2022
@veli-m-andirin veli-m-andirin deleted the issue10_autofill branch July 15, 2022 15:16
@veli-m-andirin
Copy link
Collaborator Author

Summary here.

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.

4 participants