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

[GT-184] Apply styles for the PR #464 #484

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

Sae126V
Copy link
Contributor

@Sae126V Sae126V commented Sep 19, 2023

In extend to #464 . Adding the styles PR with few minor improvements.

@Sae126V Sae126V force-pushed the GT-184-styles-Extend-API-Credential-Management-to-handle-periodic-renewal branch from 214063a to 3fc2eed Compare September 20, 2023 00:22
@Sae126V Sae126V force-pushed the GT-184-styles-Extend-API-Credential-Management-to-handle-periodic-renewal branch from 3fc2eed to 86c7ee6 Compare September 20, 2023 00:23
@Sae126V Sae126V marked this pull request as ready for review September 20, 2023 06:21
@Sae126V Sae126V requested a review from a team as a code owner September 20, 2023 06:21
Comment on lines +39 to +53
<script
type="text/javascript"
src="<?= \GocContextPath::getPath(); ?>javascript/confirm.js"
></script>

<a
onclick="return confirmSubmit()"
href="index.php?Page_Type=Delete_Site&id=<?php
echo($entityId);
?>"
>
<img
src="<?= \GocContextPath::getPath(); ?>img/trash.png"
class="trash"
/>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These split tags are a bit ugly, but not sure what the better solution is.

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