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 some numbering and formatting problems on en and ht EULAs #671

Merged
merged 5 commits into from
Apr 25, 2020

Conversation

SamMakesThings
Copy link
Contributor

Fixed the numbering so it didn't restart halfway through counting, removed the list nesting in the English EULA, and cleaned up both files a bit.

Linked issues:

None that I'm aware of.

Screenshots:

Screenshot from 2020-04-24 05-55-17

How to test

I don't think much testing is necessary since I just changed the HTML strings, but a visual scan to make sure there aren't any weird display problems in the HTML would be good

@tstirrat tstirrat merged commit eb207b1 into Path-Check:develop Apr 25, 2020
tstirrat added a commit that referenced this pull request Apr 26, 2020
* develop:
  Fix formatting EULAs (#671)
  [i18n] Don't translate terms_of_use_url (#642)
  Fix background to run to edge of screen on iOS (#682)
tstirrat added a commit that referenced this pull request Apr 28, 2020
* develop:
  Fix "plural" keys, add i18n checks for new PRs (#648)
  Auto add authorities (#587)
  Clean up more eslint issues (#678)
  Make nav back arrow color theme aware (#686)
  yarn run-ios now runs pod install (#681)
  Fix settings icon on iOS 12, replace with new icon (#687)
  Feature/creole tests (#690)
  Fix formatting EULAs (#671)
  [i18n] Don't translate terms_of_use_url (#642)
  Fix background to run to edge of screen on iOS (#682)
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