Skip to content

Conversation

touhidurabir
Copy link
Member

for #11690 .

Copy link
Contributor

@defstat defstat left a comment

Choose a reason for hiding this comment

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

@touhidurabir sorry for the delay on this.

I have no functional changes to comment, just some template description ones. Thus approving the PR.

* Copyright (c) 2000-2021 John Willinsky
* Distributed under the GNU GPL v3. For full terms see the file docs/COPYING.
*
* @brief A landing page displayed to users upon successful registration
Copy link
Contributor

Choose a reason for hiding this comment

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

Some change of @brief needed here.

{**
* templates/frontend/pages/userConfirmActivation.tpl
*
* Copyright (c) 2014-2021 Simon Fraser University
Copy link
Contributor

Choose a reason for hiding this comment

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

For new files I think we need to update the date to 2025

@@ -0,0 +1,17 @@
{extends file="layouts/backend.tpl"}
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we add a template description here?

Copy link
Member Author

Choose a reason for hiding this comment

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

right , I missed it .

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