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

[4.0] Update the help output and comments of the deleted file check to folder use #45

Closed
wants to merge 3 commits into from

Conversation

richard67
Copy link

@richard67 richard67 commented Jan 26, 2020

Pull Request for joomla#25559.

Summary of Changes

When a parameter is missing, the help message which is shown in that case still talks about commit references to be used and not folders or directories.

I've deficed to use the wording "directory" at the end because in some help messages for missing parameters it was already used like this.

Some code comments do the same.

Testing Instructions

Code review.

Expected result

Correct comments and help output.

Actual result

Misleading comments and help output.

Documentation Changes Required

None.

@wilsonge
Copy link
Owner

So the problem is this does need to move back to git commit references :) that’s the missing part of my current PR that blocks it from being merged

@richard67
Copy link
Author

@wilsonge So this PR is wrong and I shall close it? Please confirm, then I'll close.

But if so, then the help texts at following 2 places are wrong:

https://github.com/wilsonge/joomla-cms/blob/deleted_file_rework/build/deleted_file_check.php#L45

https://github.com/wilsonge/joomla-cms/blob/deleted_file_rework/build/deleted_file_check.php#L56

Or do I understand something wrong still?

@richard67 richard67 closed this Jan 28, 2020
@richard67 richard67 deleted the patch-1 branch January 28, 2020 18:33
@wilsonge
Copy link
Owner

Sorry for not replying. Yes the help texts should be updated as we work on the move back to use git properly

@richard67
Copy link
Author

Then take this PR as a reminder. Or should I make a new one here?

@wilsonge
Copy link
Owner

Can do a new one to fix that other text if you want sure

wilsonge pushed a commit that referenced this pull request Dec 3, 2020
… and clean up sub-layouts (#45)

* Separate link and button for dropdown items

- Move dropdown handling to main layout override
- Remove obsolete sub-layout overrides

* Use language string for screenreader text

* Add missing language string for frontend

* Adapt metismenu scss to changed dropdown
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