-
Notifications
You must be signed in to change notification settings - Fork 234
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
Cleanup references to Flash #163
Cleanup references to Flash #163
Conversation
Many thanks, we should have done this long ago ! |
Co-authored-by: Pierre Goiffon <[email protected]>
caedb8e
to
e0079a4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok for me, but we will need to do a team review... will keep you posted !
Will be reviewed by Romain & Denis |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would rather have 'navigate_relations' because 'navigate' is too common and grep is the solely option to connect the dots.
Or just 'relations', like you have 'details' and 'stimulus' for example? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, 'relations' is more meaningful than 'navigate', but I would prefer a more discriminant id like 'view-relations' because this will be helpfull on the long run when searching where this operation is used (and therefore perform a change or cleanup.
a37fb4b
to
2b1650c
Compare
Done |
Just made a few comment, once the one on the complementary modules is discussed and the small changes on the operation name fixed, it will be all good for me. Thanks for the cleanup @Hipska 👍 |
Co-authored-by: Molkobain <[email protected]>
All good, thanks @Hipska ! ✌ |
We got a "This branch cannot be rebased safely: Rebasing the commits of this branch on top of the base branch cannot be performed automatically as this would create a different result than a regular merge." Can you try rebasing your branch on develop? Or is it because your branch have diverged from develop? |
That's why I don't like rebase's.. I checked and don't see any merge conflicts. Is a regular merge not possible instead? |
Hi @Hipska, Guillaume had an error message but I didn't. I've just done the merge ! |
Thanks @piRGoif ! |
Hi, could you please also add the |
This was done here too |
While Flash support is deprecated, the codebase still has a lot of references to Flash.
This PR removes all the swf files and an unused swf library that get's loaded on every page. It deletes a php page with abandoned code. Since it never got referenced, it is better/safer to completely remove it. It also renames the action to load the relation graph as it still got a swf reference.