Skip to content

2 fa#3007

Open
StarmanMartin wants to merge 14 commits intomainfrom
2FA
Open

2 fa#3007
StarmanMartin wants to merge 14 commits intomainfrom
2FA

Conversation

@StarmanMartin
Copy link
Collaborator

  • rather 1-story 1-commit than sub-atomic commits

  • commit title is meaningful => git history search

  • commit description is helpful => helps the reviewer to understand the changes

  • code is up-to-date with the latest developments of the target branch (rebased to it or whatever) => ⏩-merge for linear history is favoured

  • added code is linted

  • tests are passing (at least locally): we still have some random test failure on CI. thinking of asking spec/examples.txt to be commited

  • in case the changes are visible to the end-user,  video or screenshots should be added to the PR => helps with user testing

  • testing coverage improvement is improved.

  • CHANGELOG :  add a bullet point on top (optional: reference to github issue/PR )

  • parallele PR for documentation  on docusaurus  if the feature/fix is tagged for a release

Added workflow (page/email) to enable
Added workflow (page/email) to disable
Extended Login with OTP
Restyled Setting
Composed Account & Profile, Password und Affiliations into settings
@StarmanMartin StarmanMartin requested a review from headri March 12, 2026 09:18
@github-actions
Copy link

LCOV of commit 38459e7 during CI - Ruby Lint, RSpec, and code coverage #1051

Summary coverage rate:
  lines......: 66.6% (18266 of 27447 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

@github-actions
Copy link

LCOV of commit c119b0d during CI - Ruby Lint, RSpec, and code coverage #1052

Summary coverage rate:
  lines......: 66.6% (18267 of 27446 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

@github-actions
Copy link

LCOV of commit b2a21fc during CI - Ruby Lint, RSpec, and code coverage #1055

Summary coverage rate:
  lines......: 66.6% (18266 of 27444 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

@github-actions
Copy link

LCOV of commit b142c19 during CI - Ruby Lint, RSpec, and code coverage #1059

Summary coverage rate:
  lines......: 66.5% (18266 of 27456 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

})
.catch((err) => console.error(err.response.error));
});
// http://localhost:3000/users/two_factor_auth/request_enable?jwt=eyJhbGciOiJIUzI1NiJ9.eyJ1c2VyX2lkIjo4LCJhY3Rpb24iOiJhY3RpdmF0ZV8yZmEiLCJleHAiOjE3NzMyMjM0ODN9.TlgJcZpaJ7O4Oyoyp-Ajxyv-XCa-GlQqQTbQQa2SyDg
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Remove JWT comment

@github-actions
Copy link

LCOV of commit 9cf0516 during CI - Ruby Lint, RSpec, and code coverage #1065

Summary coverage rate:
  lines......: 66.5% (18267 of 27455 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

@github-actions
Copy link

LCOV of commit 48188a5 during CI - Ruby Lint, RSpec, and code coverage #1066

Summary coverage rate:
  lines......: 66.5% (18266 of 27457 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

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.

1 participant