Rebased the diff

Updated the Gitlab Logo

Updated the Gitlab Logo

Understood the difference there, I had changed $.fn.dataTable.render.text() to a function returning an <a> tag, hence the security issues.
Thank you!

In T4246#85120, @anlambert wrote:

@anirudhlakhotia, sorry but you are wrong here, forge URLs are santized in the moderation dashboard. This is exactly the purpose of the use of the $.fn.dataTable.render.text() render function (see Security doc of datatables).

Updated Gitlab Logo

add_forge_now: Sanitized URLs on moderation dashboard
Sanitized the URLs in the Forge URL column of the datatable and added a clickable icon to open the links in a new tab

Could you please check the rendering now?
I modified it using GIMP, but added space to the image to reduce the size on the page.

Updated Gitlab Logo

Hi
Could I work on this?

add_forge_now: Add new columns in requests moderation dashboard

In D7491#201484, @vlorentz wrote:

in the API and the internal code, you should call it "last_moderator" instead of "moderator". It's fine not to mention it in the UI, though.

add_forge_now: Add new columns in requests moderation dashboard

Updating D7491: Added Moderator Name and Last Modified Time columns to the Admin Request Dashboard

Updating D7491: Added Moderator Name and Last Modified Time columns to the Admin Request Dashboard

Updating D7491: Added Moderator Name and Last Modified Time columns to the Admin Request Dashboard

I'm having some trouble understanding the behaviour of serializers:

if has_add_forge_now_permission(request.user):

add_forge_now : Added Moderator Name and Last Modified Date to Admin Request Dashboard
Related to T4071

@anlambert Thank you for the hints
@ardumont

In D7491#197616, @ardumont wrote:

(Once applied the requested changes)
Also, if you could add a screenshot of the rendering result of your diff, that'd be neat [1]

Added Moderator Name to Admin Request Dashboard

Added moderator name column to admin request dashboard

• Added moderator name column

• add_forge_now: Add hyperlinks to forge URLs in Browse Requests tab

• Added Hyperlinks to URLs in the browse requests tab

Added Moderator name column to the admin request dashboard
(sanitized URLs added)

Added Moderator name column to the admin request dashboard

Sanitized URLs, and switched to a clickable to open the link in a new tab

Added Hyperlinks to URLs in the Browse Requests Tab

• Added hyperlinks to the URLs in Browse Requests tab

@arishmauryaa I've finished the task already though :(
It's ready for a diff
If there's anything else to add, maybe you could add that?

Hey @bchauvet ,
I'd like to give this a shot.
Since the submission date is already present, will the name of the moderator suffice?

Hey @ardumont!
I'm unable to run swh-web using the docker images, more details here
Could you please help me out with this?