Skip to content

Conversation

@brooketopcoder
Copy link
Contributor

@brooketopcoder brooketopcoder commented Jun 28, 2022

This PR removes the dependency for the MFE from the Platform UI app.

In order to get this working, I had to do several things...

  • upgrade the legacy app to use the latest router library.
  • configure the app to permit files outside the src directory
  • fix lint issues
  • convert all imports to relative
  • convert all svgs to react components
  • remove the styleName attribute

This should serve locally WITHOUT the MFE...

> yarn install
> yarn start

NOTE you will need to deep-link to the dashboard. the redirects are in a diff branch.

https://local.topcoder-dev.com:3000/work/dashboard

…e of the /src directory; linting config for es; remove all references to mfe-header; remove the styleName feature; fix lint issues; convert all imports to relative; convert all SVGs to react components; #time 6h
@brooketopcoder brooketopcoder self-assigned this Jun 28, 2022
@brooketopcoder brooketopcoder changed the base branch from PROD-2194_reincarnate to master June 28, 2022 23:33
@brooketopcoder brooketopcoder changed the base branch from master to PROD-2194_reincarnate June 28, 2022 23:34
@brooketopcoder brooketopcoder changed the base branch from PROD-2194_reincarnate to PROD-2195_migrate-mfe June 28, 2022 23:34
@brooketopcoder brooketopcoder changed the base branch from PROD-2195_migrate-mfe to PROD-2194_reincarnate June 28, 2022 23:35
@brooketopcoder brooketopcoder changed the base branch from PROD-2194_reincarnate to PROD-2195_migrate-mfe June 28, 2022 23:37
Base automatically changed from PROD-2195_migrate-mfe to PROD-2195_migrate June 28, 2022 23:38
@brooketopcoder brooketopcoder changed the title PROD-2195 remove mfe -> PROD-2194_reincarnate PROD-2195 remove mfe -> PROD-2195_migrate Jun 29, 2022
@brooketopcoder brooketopcoder merged commit c9b3604 into PROD-2195_migrate Jun 29, 2022
@brooketopcoder brooketopcoder deleted the PROD-2195_remove-mfe-yipee branch June 29, 2022 21:40
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