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

Adding some contribution guidelines #277

Merged
merged 8 commits into from
Nov 27, 2023

Conversation

Shazib
Copy link
Contributor

@Shazib Shazib commented Nov 18, 2023

Adding some additional comments on making changes that could impact electron, as well as starting some stub pages for db migrations and general contribution advice.

Hopefully these two pages can be updated by others as we discover more things or can be used as launchpads for some kind of automated checks etc.

Copy link

netlify bot commented Nov 18, 2023

Deploy Preview for actualbudget-website ready!

Name Link
🔨 Latest commit 8d8d538
🔍 Latest deploy log https://app.netlify.com/sites/actualbudget-website/deploys/6563a181796cc20008962317
😎 Deploy Preview https://deploy-preview-277.www.actualbudget.org
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Shazib Hussain and others added 2 commits November 18, 2023 18:54
@Shazib
Copy link
Contributor Author

Shazib commented Nov 18, 2023

Maybe the migrations page should be a sub-section of the existing database page

This comment has been minimized.

@youngcw
Copy link
Member

youngcw commented Nov 27, 2023

@MatissJanis Does this all look good from a technical point of view? Is there anything that should be added?

@MatissJanis MatissJanis merged commit 8a90292 into actualbudget:master Nov 27, 2023
6 checks passed
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.

3 participants