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

add passbook #692

Closed
wants to merge 1 commit into from
Closed

add passbook #692

wants to merge 1 commit into from

Conversation

juancv
Copy link

@juancv juancv commented Feb 17, 2024

No description provided.

Copy link
Collaborator

@mvdkleijn mvdkleijn left a comment

Choose a reason for hiding this comment

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

What's the enhanced part of this app? I'm pretty sure passbolt doesn't have a queue or speed to show in the enhanched section.

Shouldn't this be a foundation app instead?

@mvdkleijn
Copy link
Collaborator

Also, it doesn't have an icon. Please see the wiki for details on the icon. An icon is required.

@mvdkleijn
Copy link
Collaborator

Lastly, the passbolt files don't pass the codesniffer tests.

This implies to me that you copied old code from somewhere instead of using the developer template. New PRs using the dev templates should pass the codesniffer checks.

Specifically see the code sniffer results in the failed tests.

@listiges-kaenguru
Copy link
Contributor

Added the foundation app in PR #702

@mvdkleijn
Copy link
Collaborator

Closing this in favor of #702
No offense @juancv 😄

@mvdkleijn mvdkleijn closed this Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants