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

Full order info #119

Closed
wants to merge 11 commits into from
Closed

Full order info #119

wants to merge 11 commits into from

Conversation

valentino-sm
Copy link
Contributor

@valentino-sm valentino-sm commented May 6, 2021 •
edited

  1. Fix brief text order message to user.
  2. Save order_info to worker.
  3. Save new transaction to db with order_info instead of empty row.
  4. Show order_info in full order view for admin.
  5. Hide notes if empty.

Full backward compatibility except for three lines in several language files shown for admin order views.

@Steffo99 Steffo99 self-requested a review May 6, 2021 15:54
@Steffo99 Steffo99 self-assigned this May 6, 2021
@valentino-sm valentino-sm mentioned this pull request May 6, 2021
Closed
@Steffo99
Copy link
Owner

Steffo99 commented May 6, 2021

I'm working on this.

@Steffo99
Copy link
Owner

Steffo99 commented May 6, 2021

I implemented this feature in a different way in 24d5024 :)

Thanks for the idea, I completely forgot to show that transaction data!

@Steffo99 Steffo99 closed this May 6, 2021
@Steffo99 Steffo99 added the feature A request for a new feature. label May 6, 2021
@Steffo99 Steffo99 removed their request for review May 6, 2021 23:33
@Steffo99
Copy link
Owner

Steffo99 commented May 6, 2021

Sorry for messing up your branch, IntelliJ IDEA did something weird while I was trying to merge...

You can git reflog to return to the previous state, if you wish.

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

@Steffo99 Steffo99

Labels
feature A request for a new feature.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@valentino-sm @Steffo99 @jsramirezch