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

feat(Editor): Add deprecated support #1107

Merged
merged 1 commit into from
Apr 9, 2020

Conversation

aoudiamoncef
Copy link

@aoudiamoncef aoudiamoncef commented Apr 5, 2020

Hi @EricWittmann

Please don't laugh, i' definitely not a front-end developer. Don't hesitate to give me some feedbacks about this feature.

Screenshot from 2020-04-05 19-28-15
Screenshot from 2020-04-05 19-28-38
Screenshot from 2020-04-05 19-29-02

Copy link
Member

@EricWittmann EricWittmann left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution! I would never laugh at anyone. :)

I have one small change requested, and one question for you. See comments in the PR.

@EricWittmann
Copy link
Member

Hey @aoudiamoncef the maven build is failing for this. If you get a chance can you have a look? If not I'll get to it when I can. FYI the build is less permissive than the UI dev server, so sometimes the build will fail even if everything seems to work at dev time. :)

@aoudiamoncef
Copy link
Author

Hi @EricWittmann,
I rebased from the current failing commit in master, I don't think that the PR modification could break the build.

@EricWittmann
Copy link
Member

OK I'll try it out locally.

@EricWittmann
Copy link
Member

OK I found the problem. You had removed the single-quotes from that one attribute in the template, but kept the square braces. No worries - I've fixed it and will merge soon.

@EricWittmann EricWittmann merged commit 1be1c80 into Apicurio:master Apr 9, 2020
@EricWittmann
Copy link
Member

All done, thanks for the contribution @aoudiamoncef

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