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

feat(project.new): Add the legal text and a checkbox in the second step of a project creation #9847

Closed
wants to merge 9 commits into from

Conversation

brofik
Copy link
Contributor

@brofik brofik commented Aug 16, 2023

Question Answer
Branch? master
Bug fix? no
New feature? yes
Breaking change? no
Tickets Fix #TAPC-83
License BSD 3-Clause
  • Try to keep pull requests small so they can be easily reviewed.
  • Commits are signed-off
  • Only FR translations have been updated
  • Branch is up-to-date with target branch
  • Lint has passed locally
  • Standalone app was ran and tested locally
  • Ticket reference is mentioned in linked commits (internal only)
  • Breaking change is mentioned in relevant commits

Description

Related

TAPC-83: #9779
TAPC-150 : #9831

qpavy
qpavy previously approved these changes Aug 18, 2023
JacquesLarique
JacquesLarique previously approved these changes Aug 18, 2023
anooparveti
anooparveti previously approved these changes Aug 29, 2023
JacquesLarique
JacquesLarique previously approved these changes Aug 29, 2023
qpavy
qpavy previously approved these changes Aug 29, 2023
}

displayCreateProjectButton() {
if (this.model.paymentMethod.type.isCreditCard()) {
Copy link
Contributor

@sachinrameshn sachinrameshn Sep 1, 2023

Choose a reason for hiding this comment

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

I guess we dont display this component for the card type credit card. We have another component in the same file <ovh-payment-method-integration which will display only if $ctrl.model.paymentMethod.isHandleByComponent()

sachinrameshn
sachinrameshn previously approved these changes Sep 7, 2023

hasLegalAgreements() {
return Object.values(this.model.legalAgreements).find(
(elm) => elm === true,
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
(elm) => elm === true,
(agreement) => agreement === true,

@sachinrameshn sachinrameshn force-pushed the feat/TAPC-83 branch 2 times, most recently from 1839682 to ffdf9a8 Compare September 21, 2023 12:36
@github-actions github-actions bot added the has conflicts Has conflicts to resolve before merging label Sep 27, 2023
Rafik Adiche and others added 7 commits October 17, 2023 09:29
ref: MANAGER-11567
Signed-off-by: Rafik Adiche <[email protected]>
ref: TAPC-83
Signed-off-by: Rafik Adiche <[email protected]>
ref: TAPC-150

Signed-off-by: Guillaume Hyenne <[email protected]>
ref: TAPC-83
Signed-off-by: Rafik Adiche <[email protected]>
@github-actions github-actions bot removed the has conflicts Has conflicts to resolve before merging label Oct 17, 2023
@sonarcloud
Copy link

sonarcloud bot commented Oct 17, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Contributor

This Pull Request is stale due to inactivity since 30 days. If no activity happens for 15 more days from now, it will be closed.

@github-actions github-actions bot added the stale No activity since a long time label Nov 17, 2023
Copy link
Contributor

github-actions bot commented Dec 2, 2023

This Pull Request is now closed due to inactivity.

@github-actions github-actions bot closed this Dec 2, 2023
@brofik brofik deleted the feat/TAPC-83 branch February 14, 2024 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants