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.

Sign up for GitHub

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

Jump to bottom

v1.1.0 #55

Merged
merged 22 commits into from
Jan 28, 2022
Merged

v1.1.0 #55

merged 22 commits into from
Jan 28, 2022

Conversation

kylejb
Copy link
Owner

@kylejb kylejb commented Jan 17, 2022 •
edited
Loading

The focal point of this PR is to ensure the styling of the application is maintained/improved before merging.

Potential Breaking Changes

  • Deployment: JavaScript was replaced with TypeScript.
  • CSS: node-sass was replaced with tailwindcss.

New Features

  • Begin to offload burden from React to server for some computationally intensive tasks.
  • Create Testing infrastructure.
  • Add basic tests for server and web.
  • Create CI/CD pipeline for PRs to main.

Future Plans

  • Redis support (integrated in Heroku and needs to be incorporated into codebase).
  • Refactor components into smaller reusable components with styling applied.

@kylejb
deps: bump react-script to v5 (#48)
1a0e2de
@kylejb
feat(web): refactor to TypeScript (#47)
dfe8d92 
* feat: scaffold TypeScript support with minimal typing

* feat(sighting-card): add basic TS support
@kylejb
fix(server): replace request with axios
33a145b
@kylejb
feat(server): refactor to support TypeScript
7077418
@kylejb
feat: offload client-side work to server
45f83f1 
* fix: running web in production from server
@kylejb
feat(web): replace scss with tailwindcss
212e9df 
* fix: manifest background_color

* deps: add tailwindcss library, along with devDependencies for building

* deps: remode node-sass and update dotenv

* chore(compass): remove unused scss

* feat(tailwindcss): add support for custom font families

* feat(app): refactor with tailwindcss

* feat(earth-container): refactor with tailwindcss

* feat(dropdown-container): refactor with tailwindcss

* feat(search): refactor with tailwindcss

* feat(search-container): refactor with tailwindcss

* feat(credits): refactor with tailwindcss

* feat(faq): refactor with tailwindcss

* feat(faq): replace h1 with h2

* feat(instructions): refactor with tailwindcss

* feat(notifications): refactor with tailwindcss

* feat(sighting-card): refactor to tailwindcss

* feat(splash-page): refactor with tailwindcss

* feat(splashpage): use local image

* wip(splashpage): fix for FOUC

* deps: update package-lock
@kylejb
fix(web): entry point to run tests
93aedb6
@kylejb
feat(test): scaffold server-side tests (#54)
b08f028 
* deps: add devDependencies

* feat: update package.json scripts to support CI/CD
@kylejb kylejb added enhancement New feature or request help wanted Extra attention is needed dependencies Pull requests that update a dependency file labels Jan 17, 2022
@kylejb
fix(devops): install dependencies script
f15dcec 
* feat: add deepsource support for typescript

* fix: run ci/cd only when PR is marked ready

* chore: explictly exclude node_modules as per policy rule
kylejb and others added 3 commits January 17, 2022 18:08
@kylejb
feat: add pre-commit hook to format and test code (#56)
5cfbb8f 
* chore: format all files with prettier

* chore: replace depreciated jsxBracketSameLine with bracketSameLine

* fix: npm test scripts
@kylejb
chore: address select deepsource issues
d56a2e3 
* chore(web): refactor remaining containers to TypeScript

* chore: auto-sort import statements

* chore: additional deepsource fixes
@matt-green1
refining styles
14e0997
@kylejb kylejb added the in progress Issue/PR is being worked on label Jan 22, 2022
kylejb and others added 3 commits January 22, 2022 15:34
@kylejb
deps: rebuild package-lock.json
01e8d34
@kylejb @matt-green1
feat(web/sighting-card): replace flex with grid (#58)
7dc8009 
* feat(web/sighting-card): replace flex with grid

* fix: styling to support grid

* chore: update styling

Co-authored-by: matt-green1 <60438349+matt-green1@users.noreply.github.com>
@matt-green1
refine styling on grid
f4705b7
@kylejb kylejb marked this pull request as ready for review January 24, 2022 00:17
@kylejb kylejb removed the request for review from matt-green1 January 24, 2022 00:19
@kylejb kylejb removed the help wanted Extra attention is needed label Jan 24, 2022
@kylejb kylejb temporarily deployed to sst-cicd-dev-g5jnvgaenu5dmf2sx January 24, 2022 00:54 Inactive
@kylejb kylejb temporarily deployed to sst-cicd-dev-fakttxcktipbrj26c January 24, 2022 01:08 Inactive
@kylejb kylejb temporarily deployed to sst-cicd-dev-w1ehzm1l0n8ut9nuy January 24, 2022 01:42 Inactive
@kylejb kylejb temporarily deployed to sst-cicd-dev-cwlj9k3hv7avoupmf January 24, 2022 01:50 Inactive
@kylejb kylejb temporarily deployed to sst-cicd-dev-vc1zsxrnls6ixxnqe January 24, 2022 02:04 Inactive
@kylejb kylejb temporarily deployed to sst-cicd-dev-fwqyfksq4652qnwru January 24, 2022 02:29 Inactive
@kylejb kylejb temporarily deployed to sst-cicd-dev-9jc8alhz1hkawrfyn January 24, 2022 02:32 Inactive
@kylejb kylejb temporarily deployed to sst-cicd-dev-3562mqby8rkq3us7a January 24, 2022 02:38 Inactive
@kylejb kylejb temporarily deployed to sst-cicd-dev-3562mqby8rkq3us7a January 25, 2022 01:22 Inactive
@kylejb kylejb temporarily deployed to sst-cicd-dev-3562mqby8rkq3us7a January 25, 2022 01:36 Inactive
@kylejb kylejb temporarily deployed to sst-ci-cd-dev-t70v0wya4zz32sjv January 28, 2022 01:32 Inactive
@kylejb kylejb temporarily deployed to sst-ci-cd-dev-t70v0wya4zz32sjv January 28, 2022 01:58 Inactive
@kylejb
fix(web): styling
62af786 
* fix(earth-container): cors error in safari

* fix(search-container): inconsistent width

* fix(sighting-card): grid spacing

* fix(faq): alignment and spacing of content

* fix(instructions): alignment and spacing of content

* fix(splashpage): erroneous icon suggesting missing content in chrome
@kylejb kylejb temporarily deployed to sst-ci-cd-dev-t70v0wya4zz32sjv January 28, 2022 05:55 Inactive
@kylejb kylejb temporarily deployed to sst-ci-cd-dev-t70v0wya4zz32sjv January 28, 2022 14:05 Inactive
@kylejb kylejb temporarily deployed to sst-ci-cd-dev-t70v0wya4zz32sjv January 28, 2022 22:04 Inactive
@kylejb kylejb removed the in progress Issue/PR is being worked on label Jan 28, 2022
matt-green1
matt-green1 approved these changes Jan 28, 2022
View reviewed changes
Copy link
Collaborator

@matt-green1 matt-green1 left a comment

Choose a reason for hiding this comment

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

  • revised splash text animation and placement
  • Faq/instructions formatting
  • text input formatting
  • Direction sizing

@kylejb kylejb merged commit 8d84a84 into main Jan 28, 2022
@kylejb kylejb deleted the dev branch January 28, 2022 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matt-green1 matt-green1 matt-green1 approved these changes

Assignees

@kylejb kylejb

@matt-green1 matt-green1

Labels
dependencies Pull requests that update a dependency file enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kylejb @matt-green1