Skip to content
This repository has been archived by the owner on Jul 28, 2020. It is now read-only.

Fixed issue #133, templates file upload folder selection issue. #134

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ahamed
Copy link
Contributor

@ahamed ahamed commented Dec 2, 2019

Pull Request for Issue #133 .

Summary of Changes

This pull request fixes template file upload modal's folder selection issue. And also improve scss.

Testing Instructions

Go to `Extensions > Templates > On Cassiopeia template click Edit Files > Click New File

Expected result

At the left side folder tree by clicking on a folder should select the folder and toggle its descendants.
Screenshot 2019-12-02 at 12 32 06 PM

Actual result

It reloaded the page by clicking on the folders.

Documentation Changes Required

@ahamed ahamed requested a review from wilsonge as a code owner December 2, 2019 06:41
@brianteeman
Copy link
Contributor

The changes here can not be correct. You should not have to be making changes to the upstream files when they work perfectly with atum. The fact you need to change them is indicative of a problem in the template code and not in the system code

@ahamed
Copy link
Contributor Author

ahamed commented Dec 2, 2019

We used joomla-modal Webcomponent instead of bootstrap modal and for that, we have to do this.

@brianteeman
Copy link
Contributor

great that you are using the webcomponent but that doesnt change my comment

Copy link
Contributor Author

@ahamed ahamed left a comment

Choose a reason for hiding this comment

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

Add extra newline.

@VanTrieuThanh
Copy link

I don't see what happen when click on folder as: select the folder and toggle its descendants.

@thaoredweb
Copy link

Now, I can upload a new file but could not indicate the folder inside. It is in the root @ahamed07sajeeb
b2
https://drive.google.com/file/d/1NoV3q6UX4OAqk8aIgNdnPc7NvZj5db9Y/view

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants