Skip to content

OAS Diff html report based on react with new design - #4

Merged
IgorGov merged 40 commits into
developfrom
json_objects
May 24, 2022
Merged

OAS Diff html report based on react with new design#4
IgorGov merged 40 commits into
developfrom
json_objects

Conversation

@IgorGov

@IgorGov IgorGov commented May 16, 2022

Copy link
Copy Markdown
Contributor

image

@IgorGov
IgorGov changed the base branch from main to develop May 16, 2022 11:39
@leon-up9
leon-up9 requested a review from gustavomassa May 16, 2022 15:17
@leon-up9 leon-up9 changed the title Json objects OAS Diff html report based on react with new design May 22, 2022
@leon-up9
leon-up9 marked this pull request as ready for review May 22, 2022 08:31
@leon-up9
leon-up9 requested a review from lirazyehezkel May 22, 2022 08:31
Comment thread ui/public/index.html Outdated
Comment thread ui/src/components/ChangeLog/ChangeLog.tsx Outdated
Comment thread ui/src/components/PathList/PathListItem.tsx Outdated
Comment thread ui/src/components/PathList/PathListItem.tsx Outdated
Comment thread ui/src/components/PathList/PathListItem.tsx Outdated
@IgorGov

IgorGov commented May 22, 2022

Copy link
Copy Markdown
Contributor Author

We should mention in some place that "go build" won't work (after this merge), and to build the executable should be only with "make build".
README is the place?

Comment thread ui/src/components/PathList/PathListItem.tsx Outdated
Comment thread ui/src/components/PathList/PathListItem.tsx Outdated
Comment thread ui/src/consts.ts Outdated
Comment thread ui/src/consts.ts Outdated
type EnumDictionary<T extends string | symbol | number, U> = {
[K in T]: U;
};
export const TypeCaptionDictionary: EnumDictionary<string, string> = {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

try to use https://stackoverflow.com/questions/6215349/getting-corresponding-enum-key-for-the-value-passed or other solution
for taking the enum key by value

Comment thread ui/src/consts.ts Outdated
export const deleteClass = ChangeTypeEnum.Deleted
export const infoClass = "info"

type EnumDictionary<T extends string | symbol | number, U> = {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove if not needed

@leon-up9 leon-up9 mentioned this pull request May 24, 2022
Comment thread ui/package-lock.json
"inline-chunk-html-plugin": "^1.1.1",
"react": "^18.1.0",
"react-dom": "^18.1.0",
"@mui/styles": "^5.8.0",

@IgorGov IgorGov May 24, 2022

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"make build" will generate changes in package-lock.json

@IgorGov
IgorGov merged commit 3924db5 into develop May 24, 2022
@IgorGov
IgorGov deleted the json_objects branch May 24, 2022 15:35
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.

4 participants