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.

Sign up for GitHub

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

Jump to bottom

Refactor term and conditions #807

Closed
wants to merge 13 commits into from
Closed

Refactor term and conditions #807

wants to merge 13 commits into from

Conversation

ronaldsg20
Copy link
Member 

	I've updated the Features model in order to include the enabled attribute
	as boolean and value with the required additional data for the features, terms
	and conditions in this case.

Base automatically changed from feature/2.1 to main September 3, 2024 14:23
@ronaldsg20 ronaldsg20 force-pushed the refactor/ff-terms-and-conditions branch from 323f5b8 to 8a51cf7 Compare September 4, 2024 15:37
@ronaldsg20 ronaldsg20 changed the base branch from main to feature/2.2 September 4, 2024 15:40
@ronaldsg20 ronaldsg20 marked this pull request as ready for review September 4, 2024 15:40
annipi
annipi approved these changes Sep 5, 2024
View reviewed changes
Copy link
Contributor

@annipi annipi left a comment

Choose a reason for hiding this comment

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

LGTM!

lserra-iov and others added 13 commits September 9, 2024 12:48
@lserra-iov
Remove console.logs
e1f5423
@alexjavabraz
Update package-lock.json
c751fa9
@alexjavabraz
Update package.json
0b1d689
@lserra-iov @ronaldsg20
Captcha resolver (wip)
f79dd32
@ronaldsg20
Including recaptcha for pegin and pegout form
488cc60
@ronaldsg20
Include recaptcha token on flyover pegout flow
1456e7c 
		We've updated the flyover service implementing the g-recaptcha
		in the token resolver method
@annipi @ronaldsg20
refactor: updating Account type label
baf835a
@annipi @ronaldsg20
refactor: updatinn pegin and pegout option card titles
500d01f
@annipi @alexjavabraz
fix: options pegin resize when no flyover option enabled
4c505c0
@annipi @alexjavabraz
fix: align input address
743878a
@lserra-iov @alexjavabraz
Change address placeholder
819b739
@annipi @alexjavabraz
fix: flyover pegout summary order to show rootstock to bitcoin
39f93ba
@ronaldsg20
Refactor term and conditions
87bdff4 
		I've updated the Features model in order to include the enabled attribute
		as boolean and value with the required additional data for the features, terms
		and conditions in this case.
@ronaldsg20 ronaldsg20 force-pushed the refactor/ff-terms-and-conditions branch from 8a51cf7 to 87bdff4 Compare September 18, 2024 15:17
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Sep 18, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@ronaldsg20 ronaldsg20 changed the base branch from feature/2.2 to feature/2.3 September 20, 2024 15:05
leoiovlabs
leoiovlabs approved these changes Sep 24, 2024
View reviewed changes
Copy link

@leoiovlabs leoiovlabs left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lserra-iov lserra-iov lserra-iov approved these changes

@leoiovlabs leoiovlabs leoiovlabs approved these changes

@annipi annipi annipi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ronaldsg20 @annipi @leoiovlabs @lserra-iov @alexjavabraz