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

Allow bulk upload monitors using a CSV file or JSON file. #4065

Closed
1 task done
qburst-rupesh opened this issue Nov 20, 2023 · 7 comments
Closed
1 task done

Allow bulk upload monitors using a CSV file or JSON file. #4065

qburst-rupesh opened this issue Nov 20, 2023 · 7 comments
Labels
area:api concearning the api or automation feature-request Request for new features to be added

Comments

@qburst-rupesh
Copy link

⚠️ Please verify that this feature request has NOT been suggested before.

  • I checked and didn't find similar feature request

🏷️ Feature Request Type

New Monitor

🔖 Feature description

Here, we would like to propose the addition of a new functionality, such as the ability to upload a CSV or JSON file for mass or bulk creation on additional monitors.

For example, I would like to upload 2 monitors one for https://hackernoon.com/ and second for https://www.diffchecker.com/
so there should be facility for uploading file from VueJS frontend with file validation (File type may be CSV or JSON whichever easy and compatible with our system) and after successfully submit the file new monitor should be create one for hackernoon and second for diffchecker and rest of the operation should be act as as it is like.

@louislam and all other contributers

✔️ Solution

The bulk upload monitor option is available on the right side when you click Add Monitor. The user uploads the file from there, and it is sent to the backend node.js over a socket.

It is necessary for us to read and save the file, then create the monitors using the given keys or parameters. and then unlink or delete file after our operation will be completed for avoiding junks.

❓ Alternatives

NA

📝 Additional Context

NA

@qburst-rupesh qburst-rupesh added the feature-request Request for new features to be added label Nov 20, 2023
@CommanderStorm
Copy link
Collaborator

I think this is a duplicate of #1190 #2297
If you agree, could you please close this Issue, as duplicates only create immortal zombies and are really hard to issue-manage?
If not, what makes this issue unique enough to require an additional issue? (Could this be integrated into the issue linked above?) ^^

@qburst-rupesh
Copy link
Author

@CommanderStorm No front-end code or socket code with file type inspection and file unlinking to prevent trash can be found. There are various types of monitors, such as TCP, MQTT, and https, thus it's possible that some information about which parameter to send when choosing a monitor type is lacking. Thus, it appears that the precise top-to-bottom flow is unable to identify and fully satisfy the criteria that are thought of as generic requirements.

@CommanderStorm
Copy link
Collaborator

something was lost in translation (what you wrote does not make sense to me) ^^

No front-end code or socket code [...] can be found

That is to be expected, as noted above, I think this issue is a duplicate of other issues and merging them would be benefitial

There are various types of monitors [...] thus it's possible that some information about which [monitor] parameter[s] [..] is lacking

It is unclear why this requires another issue.
A solution should be general enough to not require hardcoded columns or #3967 (comment)

the precise top-to-bottom flow is unable to identify and fully satisfy the criteria that are thought of as generic requirements

It is unclear what you mean by this statement. What makes this issue unique enough to require an additional issue?

@CommanderStorm
Copy link
Collaborator

@CommanderStorm CommanderStorm mentioned this issue Dec 1, 2023
32 tasks
@CommanderStorm CommanderStorm added the area:api concearning the api or automation label Dec 5, 2023
@CommanderStorm
Copy link
Collaborator

Duplicate of #1190

@CommanderStorm CommanderStorm marked this as a duplicate of #1190 Dec 13, 2023
@CommanderStorm CommanderStorm closed this as not planned Won't fix, can't repro, duplicate, stale Dec 13, 2023
@Vaskata84

This comment has been minimized.

@CommanderStorm
Copy link
Collaborator

CommanderStorm commented Aug 30, 2024

@Vaskata84 this issue is not implemented. No import functionality currently exists. Please see #1190

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:api concearning the api or automation feature-request Request for new features to be added
Projects
None yet
Development

No branches or pull requests

3 participants