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

set max-lenght to 2000px because it is having some issues when you ar… #116

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

Conversation

dharmendraatredpill-linpro

set max-lenght to 2000px because it is causing some UI issues when you are having more fields.
I tested and found it working fine in my case, increasing the max height it won't cause any other issues.
When you have 15 fields for a content type and you need to scroll down to see/fill fields and help text(?) labels that do not move when you scroll down.

@dharmendraatredpill-linpro
Copy link
Author

Updated code to fixed DND (drag and drop issue), Content type metadata/properties does not refresh page or sync with selected content type when moving between different folders.

…ain throw error because that file already exists
@dharmendraatredpill-linpro
Copy link
Author

Fixed on issue for alfresco community 5.2.0 when you upload same file again throw error because that file is already exists, Copied the upload.post.js file and rename its main method as it is there with upload.post.js file in uploader-plus module.

I don't know exactly this is a right way to fix this bug but my problem have been solved as tested with all use cases.
Now I can upload a file twice with same and alfresco is renaming those file names as it is doing OOTB.
I removed the import statement of upload.post.js file that I have already copied the contents from main upload.post.js file so I do not this it is required.
I tried certain approach to rename files name in uploader-plus "upload.post.js" file but did not worked for me.

Regards
Dharmendra

@softwareloop
Copy link
Owner

The incorrect content type may be related to #117 and #133

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