Skip to content

[$450] Implement admin piece in TC plugin #15

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

Closed
jmgasper opened this issue May 8, 2020 · 13 comments
Closed

[$450] Implement admin piece in TC plugin #15

jmgasper opened this issue May 8, 2020 · 13 comments
Assignees
Labels

Comments

@jmgasper
Copy link
Collaborator

jmgasper commented May 8, 2020

Required functionality

I'd like to extend the current plugin to add a new tab and / or new options to an existing tab, on the dashboard site with the required functionality for admins.

  • Admin have the ability to identify all messages from a user
  • Platform must support the ability to export data on a per-user basis
  • Must log who posted a message, date, time, and contents if text (seems like this is already done by default - I'm getting clarification)

Screen Shot 2020-05-06 at 09 41 26

Output format

The output format can be pretty raw. I don't anticipate needing to convert from what we get from the REST methods - the goal here is just to provide the data, but there are no format requirements. JSON is fine.

Discussion

As discussed in #14 , it seems like this should be doable.

@jmgasper
Copy link
Collaborator Author

jmgasper commented May 8, 2020

Contest https://www.topcoder.com/challenges/30124703 has been created for this ticket.

This is an automated message for ghostar via Topcoder X

@jmgasper
Copy link
Collaborator Author

jmgasper commented May 8, 2020

Contest https://www.topcoder.com/challenges/30124703 has been updated - it has been assigned to obog.

This is an automated message for ghostar via Topcoder X

@atelomycterus
Copy link
Collaborator

@jmgasper 1. What about images and attachments uploaded by the user to the forum? Should we export these details in a json file? Should we export images/attachments as a zip archive?
Typically, users participating in challenges do not upload personal data.
If we plan to replace the existing forum, then perhaps there willbe user support category/taas groups, where users must pass verification, for example, some customers require passport verification or so on.
Images/Attachments may also contain personal data.

  1. One json file will be created. The structure will be like this:
{
  "user": {}
  "discussions": [],
  "comments": [],
  "drafts": [],
  "conversations": [],
   .....
}

@jmgasper
Copy link
Collaborator Author

jmgasper commented May 8, 2020

@atelomycterus - I'll ask about attachments and what the requirements are there. The JSON format looks great, thanks!

@atelomycterus
Copy link
Collaborator

atelomycterus commented May 9, 2020

@jmgasper a couple of questions:

  1. Vanilla easily allows to expand dashboard screens without coping-pasting views.
    I added an export button at 'Manage Users'.
    Let me know if you are ok with UI.

image

  1. The user with permissions ('Garden.Users.Add','Garden.Users.Edit','Garden.Users.Delete') only has access to the 'Manage Users'. We can create our own permission to show/hide the export button.
    Let me know if it's required.

  2. Do we need a date range filter to export data? It might be implemented as a popup.
    Now all data is uploaded per an user. Let me know if it's required.

P.S. The pull request - #16
Export data (user, discussions, comments, conversations, messages, drafts, IPs) was implemented.
Example jsons:
export-reports.zip

Open questions:

  • export images/attachments
  • 'forgotten' rights

@jmgasper
Copy link
Collaborator Author

@atelomycterus - Sorry for the delayed response here:

  1. UI Looks fine, thanks
  2. No, this will only be used by admins, so that list of permissions is fine.
  3. No, I don't think we need a date range yet. If we do, we'll create a new ticket for that, thanks.

For now, don't worry about exporting images / attachments or the forgotten rights, thanks.

@atelomycterus
Copy link
Collaborator

@jmgasper Thanks for details. Please apply PR - #16.

@jmgasper
Copy link
Collaborator Author

Looks good. I'll get feedback from Topcoder to see if this meets their needs.

@jmgasper
Copy link
Collaborator Author

Will close this once I get final clarification on a billing account change. There was a problem here that was still being resolved, sorry.

@jmgasper
Copy link
Collaborator Author

@atelomycterus - Sorry for the delay, still waiting on further clarifications from Topcoder RE: billing account.

@jmgasper
Copy link
Collaborator Author

Payment failed: [500]: Failed to get billing detail for the project.

This is an automated message for ghostar via Topcoder X

@jmgasper
Copy link
Collaborator Author

jmgasper commented Jun 1, 2020

This has been paid

@jmgasper jmgasper closed this as completed Jun 1, 2020
@jmgasper
Copy link
Collaborator Author

jmgasper commented Jun 1, 2020

This ticket was not processed for payment. If you would like to process it for payment, please reopen it, add the tcx_FixAccepted label, and then close it again

This is an automated message for ghostar via Topcoder X

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

No branches or pull requests

2 participants