-
-
Notifications
You must be signed in to change notification settings - Fork 46
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
Feat Remove Microfrontend #831
Conversation
…to feat-projectinfo-bug
removed fmtm_map micro frontend from docker and github actions
Awesome, so the microfrontend has been refactored to a single frontend now? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great refactor - looks good to me - thanks for checking 👍
yeah just took some time to fix it cause it was making so much problem for any other contributors to work on frontend now it will be quite easy @spwoodcock |
Great idea to make the code similar and easier to maintain ! |
Just found a build issue on fmtm_main will look into it @spwoodcock |
The deploy seemed to go ok after I just tweaked dockerfile.prod. Is the error preventing a build for you? |
I have removed fmtm_map micro frontend from docker and github actions please verify it @spwoodcock so that it wont break production and development and i have tested it in my local it worked fine.
Btw this PR have also added a feature onto it but you dont have to look all the changes you just have to look onto CONFIGS,DOCKER FILES, GITHUB ACTIONS FILE and ENV.