Skip to content

Comments

Move OC_Mount_Config to proper classname and remove OC::$CLASSPATH us…#21785

Merged
MorrisJobke merged 1 commit intomasterfrom
techdebt/noid/remove-classmap-in-files_external
Jul 13, 2020
Merged

Move OC_Mount_Config to proper classname and remove OC::$CLASSPATH us…#21785
MorrisJobke merged 1 commit intomasterfrom
techdebt/noid/remove-classmap-in-files_external

Conversation

@MorrisJobke
Copy link
Member

…age in files_external

@MorrisJobke MorrisJobke added this to the Nextcloud 20 milestone Jul 9, 2020
@MorrisJobke
Copy link
Member Author

Plus those 3 apps use it as well, but it's private namespace anyways:

Bildschirmfoto 2020-07-09 um 23 39 00

@MorrisJobke MorrisJobke force-pushed the techdebt/noid/remove-classmap-in-files_external branch from eabb916 to 47aa289 Compare July 9, 2020 21:45
@MorrisJobke MorrisJobke force-pushed the techdebt/noid/remove-classmap-in-files_external branch from 47aa289 to 065ef14 Compare July 9, 2020 22:03
@rullzer rullzer requested a review from icewind1991 July 10, 2020 09:34
@MorrisJobke
Copy link
Member Author

Plus those 3 apps use it as well, but it's private namespace anyways:

Bildschirmfoto 2020-07-09 um 23 39 00

I notified the authors of all three apps to update their apps.

Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

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

🙈 🐘

@MorrisJobke MorrisJobke merged commit cbbbbee into master Jul 13, 2020
@MorrisJobke MorrisJobke deleted the techdebt/noid/remove-classmap-in-files_external branch July 13, 2020 14:47
@MorrisJobke
Copy link
Member Author

Let's put it also in #20953

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants