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

File depot edit Page not displaying all the options #8159

Closed
akhilkr128 opened this issue Mar 4, 2022 · 12 comments · May be fixed by #8235
Closed

File depot edit Page not displaying all the options #8159

akhilkr128 opened this issue Mar 4, 2022 · 12 comments · May be fixed by #8235
Assignees

Comments

@akhilkr128
Copy link
Member

Page Navigation

Settings -> Application Settings -> Diagnostics -> Select Server -> Collect logs -> edit

Validate option for credentials as well as edit properties options are missing on the page.

Screenshot 2022-03-04 at 11 41 43 PM

@MelsHyrule
Copy link
Member

@akhilkr128 are there drop down options missing or do we want to introduce new properties/options to edit?

@kbrock
Copy link
Member

kbrock commented Mar 11, 2022

@MelsHyrule The concern is there are text boxes that just don't show up. So there was no way to create a file depot without an error for missing fields

@akhilkr128
Copy link
Member Author

The old versions will show like this for FTP and samba.
Screenshot 2022-03-15 at 12 41 23 PM
Screenshot 2022-03-15 at 12 41 04 PM

@kavyanekkalapu
Copy link
Member

@akhilkr128 Do we have api for this so that we can do new form in react?

@akhilkr128
Copy link
Member Author

No, we don't have API in place. The API request is already in place here. A discussion on chopping this option also happening link.
cc: @Fryguy

@kavyanekkalapu
Copy link
Member

@Fryguy @akhilkr128 If it is on chopping block, do we have to fix this form?

@akhilkr128
Copy link
Member Author

@kavyanekkalapu We haven't confirmed the alternative way to collect the logs, better to fix this one in my opinion.
cc: @Fryguy

@kavyanekkalapu
Copy link
Member

@akhilkr128 This bug is like working on a new form. I see that api issue is assigned to you. When you have time, Could you add api end point for this? I would prefer writing this form in react than current implementation.

@akhilkr128
Copy link
Member Author

That task is on hold because we are planning for a different approach and chop the feature itself. Converting this to react will be a waste of time if we eliminate this feature.

@kavyanekkalapu
Copy link
Member

kavyanekkalapu commented Mar 22, 2022

@akhilkr128 At present we have only one field in form. we have to add 5-6 fields. so, it is almost like adding a new form to UI. Once jason is back, we can have a call to discuss an alternative approch and i would prefer doing that instead of fixing this one.

@miq-bot
Copy link
Member

miq-bot commented Feb 27, 2023

This issue has been automatically marked as stale because it has not been updated for at least 3 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.

@miq-bot miq-bot added the stale label Feb 27, 2023
@miq-bot miq-bot closed this as completed May 29, 2023
@miq-bot
Copy link
Member

miq-bot commented May 29, 2023

This issue has been automatically closed because it has not been updated for at least 3 months.

Feel free to reopen this issue if this issue is still valid.

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.

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

Successfully merging a pull request may close this issue.

6 participants