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

Ariza front #1

Open
wants to merge 22 commits into
base: develop
Choose a base branch
from

Conversation

howmarketing
Copy link

@howmarketing howmarketing commented Nov 22, 2021

First base necessary changes to be able to build the application such as dev environment or/and basic production build with corresponds typescripts checks.

These changes are just part of a several hours working on typescript compiler configs, to get to match local IDE eslint display`s error and warnings as the runtime build application.

Such hours of effort in this type of configuration was really necessary, as many components construction type errors and conflict between dependent packages became invisible when the application was deployed, breaking features in several places in it, not having an automatic signal to show-up for us where the break are, such problems were only could be founded after some kinds of commits works, turning the project into a refactory first.

The upcoming pull request orders (before 4:00pm), are, probably will be just removing trash, what already are in progress. But, right now, the current deploy without these trash, it`s breaked and could not be online until some data trafic issues between components can be fixed (Already working on it, and it could be doned about 17:00pm).

…pendencies abrout eslint, prettier, babel, etc. And changin typescript version
…omponent. All oters detailed svgs must to be provided from here
…HERE. Some configs was hardcode to package react webpack config file and pancakeswap eslint config file, becouse the amount of conflict versions was taking so long to figure out how to do a safe packages update, and it was so much harder then just make these changes.
… (checkJs) propperty, to make development must equal as possible with error reporting comparing to production runtime deploy
…ilded in development environment as the production to.
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.

1 participant