Skip to content

Table buttons#2

Open
allysonmking wants to merge 38 commits into
allysonmking:table-updatefrom
OlinZ:table-buttons
Open

Table buttons#2
allysonmking wants to merge 38 commits into
allysonmking:table-updatefrom
OlinZ:table-buttons

Conversation

@allysonmking

Copy link
Copy Markdown
Owner

No description provided.

allysonmking and others added 30 commits March 10, 2021 17:16
…mericanAirlines#291)

* Add section to README for env variables AmericanAirlines#263

* Update README.md

* update Authors.md

* Update README.md

* Update Authors.md

Co-authored by Olin Zhou <42856711+OlinZ@users.noreply.github.com>
* ran npm audit fix

* updated authors

* updated authors

* updated jest, ts-jest, and typescript versions

Co-authored-by: Turner Levey <20231367+turnerlevey@users.noreply.github.com>
…nes#292)

* add string dictionary update

* update authors

* update naming convention to start with lowercase

Co-authored-by: Yun Phelps <36348871+YunPhelps@users.noreply.github.com>
* Update README.md

* Update README.md

* Remove ngrok from deps and scripts

* Update README.md

Co-authored-by: Spencer Kaiser <SpencerKaiser@users.noreply.github.com>

Co-authored-by: Spencer Kaiser <SpencerKaiser@users.noreply.github.com>
* Fix lots of stuff.

* Linting
* Fix lots of stuff.

* make column type simple-json. create getValueAs methods.

* Fix return value types

* Fix return value types

* Update column to jsonb. Create migration

* linting

* PR Updates and lint
* Update coverage thresholds

* Update coverage thresholds
…lines#302)

* added command aliases and made commands case insensitive

* updated generic info and hopefully fixed linting errors

* Edited generic info

* removed redundant test when checking case sensitivity and aliases

* added IStringDictionary.ts and StringDictionary.ts to pass build tests

* added registerTeam.ts and supportRequest.ts to fix project coverage

* added all changed files after syncing branch to not lose coverage

* used Stringdictionary, changed readTrigger to displayTrigger, added tests

* update index.test.ts

* updated index.test.ts

* removed console.log from help.ts and unecessary import statement in index.test.ts

* removed tests that checked if the correct object was passed

* changed message type of last test in index.test.ts to dm

Co-authored by: Tatum Hutton <78566681+tatumwashere@users.noreply.github.com>
Bumps [y18n](https://github.com/yargs/y18n) from 4.0.0 to 4.0.1.
- [Release notes](https://github.com/yargs/y18n/releases)
- [Changelog](https://github.com/yargs/y18n/blob/master/CHANGELOG.md)
- [Commits](https://github.com/yargs/y18n/commits)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Spencer Kaiser <SpencerKaiser@users.noreply.github.com>
…icanAirlines#335)

* Fixing issues #:
Eslint is not checking tsx files AmericanAirlines#217
Added "tsx" & "jsx" to the eslint accepted extensions
Added "jsx: preserved" to allow tsx translation to jsx without error
Fixed typing error in nodemonConfig delay

* Fixing issues #:
Eslint is not checking tsx files AmericanAirlines#217
Added "tsx" & "jsx" to the eslint accepted extensions
Added "jsx: preserved" to allow tsx translation to jsx without error
Fixed typing error in nodemonConfig delay
Fixed syntax to be Github friendly
Added eslintIgnore to remove the build files from linting

* Fixing issues #:
Eslint is not checking tsx files AmericanAirlines#217
Added "tsx" & "jsx" to the eslint accepted extensions
Added "jsx: preserved" to allow tsx translation to jsx without error
Fixed typing error in nodemonConfig delay
Fixed syntax to be Github friendly
Added eslintIgnore to remove the build files from linting
Adding my name to the Authors.md file
Migrating all eslint configs to the package.json file for consolidation

* Fixing issues #:
Eslint is not checking tsx files AmericanAirlines#217
Added "tsx" & "jsx" to the eslint accepted extensions
Added "jsx: preserved" to allow tsx translation to jsx without error
Fixed typing error in nodemonConfig delay
Fixed syntax to be Github friendly
Added eslintIgnore to remove the build files from linting
Adding my name to the Authors.md file

* Fixing issues #:
Eslint is not checking tsx files AmericanAirlines#217
Added "tsx" & "jsx" to the eslint accepted extensions
Added "jsx: preserved" to allow tsx translation to jsx without error
Fixed typing error in nodemonConfig delay
Fixed syntax to be Github friendly
Added eslintIgnore to remove the build files from linting
Adding my name to the Authors.md file
Adding back .eslintrc.js

* Fixing issues #:
Eslint is not checking tsx files AmericanAirlines#217
Added "tsx" & "jsx" to the eslint accepted extensions
Added "jsx: preserved" to allow tsx translation to jsx without error
Fixed typing error in nodemonConfig delay
Fixed syntax to be Github friendly
Added eslintIgnore to remove the build files from linting
Adding my name to the Authors.md file
Adding back .eslintrc.js
Removing .eslintignore file
Switching linting to only capture files within src directory
Removing unneeded tsconfig

* Apply suggestions from code review

Co-authored-by: Spencer Kaiser <SpencerKaiser@users.noreply.github.com>

* Update tsconfig.json

Co-authored-by: Spencer Kaiser <SpencerKaiser@users.noreply.github.com>
turnerlevey and others added 8 commits April 1, 2021 12:07
* fixed express bug

* ran 'npm i' to modify package-lock.json
AmericanAirlines#318)

* replaced admin/support/judge secret access from .env with the database

* changes made during our meeting with spencer

* fixed the support and judge pages issue

* updated tests in requireAuth.test.ts to use Config findOne

* made sure adminSecret.value is a string in index.ts

* added app.use in app.ts

* mocked requireAuth in team.test.ts

* removed if block in support/secret.tsx

Co-authored-by: Spencer Kaiser <SpencerKaiser@users.noreply.github.com>
* added cookie-parser in app.ts

* Update src/app.ts

Co-authored-by: John Kahn <7807353+johnkahn@users.noreply.github.com>
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.

9 participants