Skip to content

Mejoras de visualizacion de menus y traducciones#10

Open
xaviku91 wants to merge 1 commit intomainfrom
dev/mejorasmenu
Open

Mejoras de visualizacion de menus y traducciones#10
xaviku91 wants to merge 1 commit intomainfrom
dev/mejorasmenu

Conversation

@xaviku91
Copy link
Collaborator

No description provided.

Copy link
Owner

@Warrio111 Warrio111 left a comment

Choose a reason for hiding this comment

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

Fixear los test aunque no tiene mucho sentido ya que en el producto 4 estas clases del modelo desaparecen

@@ -76,13 +76,14 @@ public void setPreparationTimeMinutes(int preparationTimeMinutes) {

@Override
public String toString() {
Copy link
Owner

Choose a reason for hiding this comment

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

Rompe Lost Test

@@ -80,14 +84,15 @@ public void setShippingDiscount(double shippingDiscount) {

@Override
public String toString() {
Copy link
Owner

Choose a reason for hiding this comment

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

Rompe Lost Test

@@ -87,18 +88,21 @@ public boolean isCancelable(LocalDateTime currentTime) {

@Override
public String toString() {
Copy link
Owner

Choose a reason for hiding this comment

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

Rompe Lost Test

@@ -22,14 +23,16 @@ public double shippingDiscount() {

@Override
public String toString() {
Copy link
Owner

Choose a reason for hiding this comment

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

Rompe Lost Test

@@ -13,6 +13,7 @@ public class ManageOS {
private final Controller controller;
private final Scanner scanner;
Copy link
Owner

Choose a reason for hiding this comment

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

Ya he integrado en la main todo de esta clase

@@ -22,12 +22,14 @@ public double shippingDiscount() {

@Override
Copy link
Owner

Choose a reason for hiding this comment

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

Rompe Lost Test

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