Skip to content

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
Copy Markdown
Member

…age in files_external

@MorrisJobke MorrisJobke added 3. to review Waiting for reviews technical debt 🧱 🤔🚀 labels Jul 9, 2020
@MorrisJobke MorrisJobke added this to the Nextcloud 20 milestone Jul 9, 2020
@MorrisJobke
Copy link
Copy Markdown
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
…age in files_external

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

3. to review Waiting for reviews technical debt 🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants