Skip to content
This repository has been archived by the owner on Jul 25, 2024. It is now read-only.

refactor(issue-338): Miscellaneous chores for lit app #356

Merged
merged 6 commits into from
Apr 21, 2022

Conversation

pattishin
Copy link
Collaborator

@pattishin pattishin commented Apr 18, 2022

To verify locally:

Resolves #333 & #338

  • Replace string in line 26 with your api container url (experimental/client-app/server/proxy.js)
npm i
npm run build
npm run dev

Additional chore items include:

  • Removing axios and using native browser fetch api (recommended by lit team)
  • Refactoring main store connection from campaign-list component to app-dashboard container.
  • Refactoring out action related items into their own folder. Separate them from reducers
  • Adding in ability to configure env variable and toggle between flask proxy api (../website/views/routes.py) vs node proxy (server/proxy.js). See web-dev-server.config.mjs

pattishin and others added 5 commits April 6, 2022 20:03
feat: configurable NODE_ENV

feat: adding config files to retrieve env vars

chore: minor clean up

refactor: allowing parallel processes to run

chore: clean up

chore: clean up
chore: clean up
@pattishin pattishin added the tech: lit Related to the experimental lit frontend application label Apr 18, 2022
@pattishin pattishin requested a review from a team as a code owner April 18, 2022 17:04
@pattishin pattishin merged commit ebf6a6a into main Apr 21, 2022
@pattishin pattishin deleted the chore/issue-338 branch April 21, 2022 23:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
tech: lit Related to the experimental lit frontend application
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat(lit-frontend): (Story) Miscellaneous fixes
2 participants