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

Minor rewording and grammar fixes #173

Merged
merged 1 commit into from
Sep 19, 2023

Conversation

meenbeese
Copy link
Collaborator

@meenbeese meenbeese commented Sep 17, 2023

This PR spun off from #169 to:

  • Correct some minor grammar mistakes
  • Update the click wording to be more personal
  • Update the GH issue count to the current one

To preview the changes: https://a0bf1bbd.organicmaps-github-io.pages.dev/
So I was able to get a preview right from this PR, how did that work?

Copy link
Member

@biodranik biodranik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@meenbeese meenbeese temporarily deployed to production September 19, 2023 20:40 — with GitHub Actions Inactive
@biodranik
Copy link
Member

We can provide a write access to the repo for active contributors, so it would be easier to preview deployments automatically.

@biodranik biodranik merged commit 8c6b34f into organicmaps:master Sep 19, 2023
2 checks passed
@meenbeese meenbeese deleted the minor-rewording branch September 19, 2023 21:15
@meenbeese
Copy link
Collaborator Author

We can provide a write access to the repo for active contributors, so it would be easier to preview deployments automatically.

Yes, I would appreciate that as it would be much easier to actually focus on the changes instead of dealing with preview links :/

@biodranik
Copy link
Member

@meenbeese after accepting invitation you can push branches to the main repo and easily test deployments )

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.

2 participants