-
Notifications
You must be signed in to change notification settings - Fork 6
Reports and Service year #373
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -18,7 +18,7 @@ Terminating any session will remove your Organized account from that device or b | |||||
|
|
||||||
| Two-factor authentication (2FA) boosts your account's security. You'll need to enter a one-time code from a mobile app, such as Google Authenticator or Microsoft Authenticator. To begin, click on "Enable 2FA." If you ever want to, you can turn it off later. However, we recommend appointed brothers to have 2FA turned on. | ||||||
|
|
||||||
| #### To set up the 2FA, simply activate the toggle and follow the instructions in the app. It's just two simple steps: | ||||||
| ### To set up the 2FA, simply activate the toggle and follow the instructions in the app. It's just two simple steps: | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Remove the trailing colon from the heading. The heading at line 21 has a trailing colon, which is not necessary and should be removed to follow the best practices for markdown. Apply this diff to remove the trailing colon: -### To set up the 2FA, simply activate the toggle and follow the instructions in the app. It's just two simple steps:
+### To set up the 2FA, simply activate the toggle and follow the instructions in the app. It's just two simple stepsCommittable suggestion
Suggested change
ToolsMarkdownlint
|
||||||
|
|
||||||
| 1. Set up 2FA (two-factor authentication) on your device. You can choose from the following methods: | ||||||
|
|
||||||
|
|
||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! But remove the trailing punctuation in the heading.
The change in the heading from "Add theocratic assignments field to my report" to "Add credit field to my report" is approved, as it is consistent with the AI-generated summary.
However, the static analysis tool has correctly flagged a trailing punctuation issue in the heading.
Apply this diff to remove the trailing punctuation:
Committable suggestion
Tools
Markdownlint