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

Alphanumeric passcodes #1257

Merged
merged 24 commits into from
Sep 30, 2023
Merged

Alphanumeric passcodes #1257

merged 24 commits into from
Sep 30, 2023

Conversation

mrose17
Copy link
Member

@mrose17 mrose17 commented Sep 30, 2023

No description provided.

renovate bot and others added 24 commits August 8, 2023 09:06
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Remove check for Brave browser when creating a room.
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Change to make `?sol=y` as the default.  In a few weeks, we'll remove the query flag altogether.
as noted by @kdenhartog, this file should be sourced from talk.brave.com
* address exclusion

* address exclusion

* address exclusion

* address exclusion

* minor touch-ups

1. restore EOL on last line of modified files.
2. use simplified Unicode \s in regex

* fix allow/deny address examples, update comments on query parameters

* Update ExceptionListPanel.tsx

courtesy of `npm run lint`

* Sync with `main` package*.json

* Update use-params.ts

Fix conflict from #1219

---------

Co-authored-by: Marshall T. Rose <mrose17@gmail.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
1. Add an `npm run check` command that runs all of the commonly failing checks: eslint, tsc, prettier and audit.

2. Run `check` as part of the CI build

3. Ensure linter and prettier run on the pre-commit hook

4. Fix the linter and formatting errors that had crept in while these checks were not enforced
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Update dependency prettier to v3

* Update formatting based on prettier v3

- v3 makes 'trailingComma' default to true (see [release notes](https://prettier.io/blog/2023/07/05/3.0.0.html#change-the-default-value-for-trailingcomma-to-all-11479httpsgithubcomprettierprettierpull11479-by-fiskerhttpsgithubcomfisker-13143httpsgithubcomprettierprettierpull13143-by-sosukesuzukihttpsgithubcomsosukesuzuki))
- v3 can also cause failures with it's automatic jest integration, just disable this in jest.config.js since we reformat on commit anyway

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Graham Tackley <gtackley@brave.com>
We used to rely on `eslint-config-react-app` to configure some aspects of eslint. This is the full-service config used by "Create React App". However it brings in other dependencies such as (an old version of) eslint itself. It also is much less maintained than it used to be, as many projects (including talk!) have moved away from using create-react-app.

Use the react specific plugins directly.

As part of this, eslint-plugin-react doesn't really like the style of declaring react functional components using `React.FC`. This is a bit of an outdated style anyway, and it's usually clearer just to create functions with explicit types. Since this is also how most react documentation sites do it, I've removed all uses of the `React.FC` type.

Co-authored-by: Marshall T. Rose <mrose17@gmail.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Marshall T. Rose <mrose17@gmail.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@mrose17 mrose17 merged commit 4a8b2ab into dev3 Sep 30, 2023
2 checks passed
@mrose17 mrose17 deleted the alphanumeric-passcodes branch October 2, 2023 16:10
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.

4 participants