inspectIT / inspectit-ocelot

inspectIT Ocelot - Java agent for collecting application performance, tracing and behavior data
http://www.inspectit.rocks/
Apache License 2.0
204 stars 69 forks source link

Closes #1506, #1452 - Update Primereact to version 8 #1517

Closed salim-1997 closed 1 year ago

salim-1997 commented 2 years ago

Closes #1506 #1452


This change is Reviewable

danipaniii commented 2 years ago

At the moment there are two more warnings left, I am not able to resolve at the moment.

setCapture_deprecated_problem

In our files we don't use any of those methods setCapture or releaseCaputre, maybe we are using a library with a text area editor component, that still uses those deprecated methods. At least that's my guess for now. Since we want to implement a multiple tab editor in the future and therefore using a new primereact component, I would look over that warning at the moment.

weird_next_issue

This error gets generated through files that are being created by next. I couldn't find anything on the internet about this issue. Still I couldn't notice any undesired behavior of the UI so far.

heiko-holz commented 1 year ago

I think I found another error message:

when left-clicking on a configuration file, I see the following in the console:

``Warning: This synthetic event is reused for performance reasons. If you're seeing this, you're accessing the propertypageY` on a released/nullified synthetic event. This is set to null. If you must keep the original synthetic event around, use event.persist(). See https://fb.me/react-event-pooling for more information. in Transition (created by CSSTransition) in CSSTransition (created by CSSTransition) in CSSTransition (created by ForwardRef) in Unknown in Unknown (created by ForwardRef) in ForwardRef in ForwardRef (at FileTree.js:192) in div (at FileTree.js:191) in div (at FileTree.js:148) in FileTree (created by Connect(FileTree)) in Connect(FileTree) (at ConfigurationView.js:226) in div (at ConfigurationView.js:215) in div (at ConfigurationView.js:188) in ConfigurationView (created by Connect(ConfigurationView)) in Connect(ConfigurationView) (at configuration.js:19) in div (at MainLayout.js:22) in div (at MainLayout.js:10) in MainLayout (at configuration.js:14) in ConfigurationPage (at _app.js:54) in AuthenticationRouter (created by Connect(AuthenticationRouter)) in Connect(AuthenticationRouter) (created by withRouter(Connect(AuthenticationRouter))) in withRouter(Connect(AuthenticationRouter)) (at _app.js:53) in UnsavedChangesGate (created by Connect(UnsavedChangesGate)) in Connect(UnsavedChangesGate) (at _app.js:52) in NotificationHandler (created by Connect(NotificationHandler)) in Connect(NotificationHandler) (at _app.js:51) in PersistGate (at _app.js:50) in Provider (at _app.js:49) in OcelotConfigurationUI (at with-redux-store.js:53) in AppWithRedux in ErrorBoundary (created by ReactDevOverlay) in ReactDevOverlay (created by Container) in Container (created by AppContainer) in AppContainer in Root



![image](https://user-images.githubusercontent.com/12088200/201040226-777c33ee-08d8-4270-9b47-cf48c6bff193.png)
danipaniii commented 1 year ago

While working on another issue, I noticed that this error message:

weird_next_issue

Occurs especially with empty .yml files. When the .yml files aren't empty this message doesn't show. Maybe someone can make a similar observation.

heiko-holz commented 1 year ago

@danipaniii:

I receive the same exception when I click on empty files:

GEThttp://localhost:3000/_next/8f7cd49ef21ef2849cb21408fd2cef2e.js
[HTTP/1.1 404 Not Found 15ms]

Loading script from “http://localhost:3000/_next/8f7cd49ef21ef2849cb21408fd2cef2e.js” with importScripts() was blocked because of a disallowed MIME type (“text/html”).

However, the following warning message is triggered for each file when I right-click on it: image

heiko-holz commented 1 year ago

I think everything is fixed now. However, I'd like to have a final chat with @danipaniii to discuss if this is really the case :)