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.

Sign up for GitHub

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

Jump to bottom

Implement URL Management Feature in Universal Migration Tool #31

Open
wants to merge 2 commits into
base: react
Choose a base branch
from
Open

Implement URL Management Feature in Universal Migration Tool #31

wants to merge 2 commits into from

Commits on May 11, 2023

  1. Rough implementation of technical coding assignment: 

       - The client side fetch requests seem to be mapping to a fallback action and returning the jsp xml instead of expected json, even after updating the method and name attributes. I Plan on continue working on a solution, after getting some rest and coming back with a fresh set of eyes! :)
    @wgeorge1990
    wgeorge1990 committed May 11, 2023
    Configuration menu
    Copy the full SHA
    22ab533 View commit details
    Browse the repository at this point in the history

  2. Update ProjectPropertiesAction.java method updateUrl to split two url… 

    …s using a space instead of a comma, and then trimming the whitespace on either side of each url to avoid errors in url parsing. TODO: add urlencoding for safer handling down the line.
    @wgeorge1990
    wgeorge1990 committed May 11, 2023
    Configuration menu
    Copy the full SHA
    00ca00f View commit details
    Browse the repository at this point in the history