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

Vite3 rebase dec14 unsquashed #11353

Draft
wants to merge 15 commits into
base: develop
Choose a base branch
from

Conversation

happythenewsad
Copy link
Contributor

This is just for reference; it will never be merged.

Peter Kong and others added 15 commits December 6, 2023 16:37
This commit contains all vite work as of today:
- vite generally serving the entire app without error
- all jests passing
- reversion to using process.env instead of import.meta.env

The unsquashed git history may be accessed via the vite3-june14-bak branch.
Latest commit in this squash: ba9fd

re-introduce vite shim, which was clobbered during a recent rebase

Vite3 prod build (#10896)

* update lockfiles

temporarily deleting entrypoints for debugging

Revert "temporarily deleting entrypoints for debugging"

This reverts commit d26e9a17aa82b9fd83f557e063cd376716cc5b7c.

removing comments

include airbyte script from develop branch

* our build doesn't support brackets in react components, so re-syntaxing the code

* move from cjs to esm syntax in particular file to fix build

* this config has a working build

* remove console log

* parameterize env vars

---------

Co-authored-by: Peter Kong <[email protected]>

remove webpack/ and webpacker.yml

comment out initializers/react_on_rails.rb

- removed webpacker gem \n - removed webpacker binstubs \n - removed webpack specific scripts in package.json

import vairables in cms_table.scss

Revert "import vairables in cms_table.scss"

This reverts commit 80ea53b.

remove cms.css from config.assets.precompile

removing obsolete Procfiles and webpack configs

Procfile config change; react_on_rails initializer config change

remove print statements

pin vcr gem; remove commented statements in erb file

CMS_URL -> QUILL_CMS

whitespace

freshen package.json

update package-locks

update Gemfiles

fix single spec regression

remove referenced to deleted files edit-assigned-students, final_score_page

fix SegmentAnalytics syntax issue, remove old css file reference, to get prod build working again

update snapshots

update snap

revert bin/bundle changes

lint

relocking Gemfile after bundle lock --add-platform x86_64-linux

silence system spec that depends on webpacker environment

add modified bundler binstub so that local env starts

Gemfile.lock updates

bundle lock --add-platform x86_64-linux

removing require statements from js files

check in new package lockfiles
add evidence scss entrypoint

removing linux platform from lockfile

adding mini_racer gem

delete unused webpacker-dependent specfile; linting

add ulimit to avoid too many files error during local serving

update moment import syntax to avoid 'moment is not a function' errors

update setupTets, tsconfig so that momentjs plays nice with jest

fix failing node specs, part 1

fix failing node specs, part 2

removing jest mock from setupTests

more jest fixes

concert C = require( statements to import statements

convert more require statements to import

Pin webmock (#11300)

fix test (#11302)

fix activity assignment flow null value bug

fix mock- and snapshot- related jest failures

fix eslint

update snapshots, remove one-off issue in conceptResults.test.ts

update jest snapshots

fix more jests

fix Pusher import statement
* remove home import from teacher bundle

* add scoring updates card to scss index

* import home file in the right place in application.scss

* fix bug where activity counts aren't showing up

* add back header margins to style guide

* fix grade level filters file

* add update-assigned-student scss to pages/index

* move locker into staff bundle so it can get the right css

* fix evidence css

* add proofreader scss entrypoint

* fix tests

* one more test

* remove require in favor of regular import

* fix merge conflict
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.

2 participants