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

Mint tax and cash #374

Merged
merged 1,039 commits into from Apr 10, 2024
Merged

Mint tax and cash #374

merged 1,039 commits into from Apr 10, 2024

Conversation

noahwc
Copy link
Contributor

@noahwc noahwc commented Jan 22, 2024 •
edited by 00salmon

Description of the change

Mint component updates for tax and cash projects.

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation or Development tools (readme, specs, tests, code formatting)

Links

Checklists

Development

  • Prettier was run (if applicable)
  • The behaviour changes in the pull request are covered by specs
  • All tests related to the changed code pass in development

Paperwork

  • This pull request has a descriptive title and information useful to a reviewer
  • This pull request has a Jira number
  • This pull request has a Process.st launch checklist

Code review

  • Changes have been reviewed by at least one other engineer
  • Security impacts of this change have been considered

Ultha258 and others added 24 commits April 2, 2024 20:26
@Ultha258
Spec was missing the 'steps' column
f9a63fb
@00salmon
bump component boilerplate
0e0d0b6
@00salmon
Merge branch 'mint-tax-and-cash' of https://github.com/saasquatch/pro…
f4281f9 
…gram-tools into mint-tax-and-cash
@00salmon
publish
0bb5ba6
@AndiLuo
Fix broken dashboard story
4411916
@00salmon
cleanup/rearrange
77ea50f
@zachharrison
cleanup index file
3a1ea12
@00salmon
more cleanup
24ca50f
@00salmon
Merge branch 'mint-tax-and-cash' of https://github.com/saasquatch/pro…
3198437 
…gram-tools into mint-tax-and-cash
@Locrian24
add check for valid tax document before step 3
0524805
@Locrian24
Merge branch 'mint-tax-and-cash' of github.com:saasquatch/program-too…
16c49e1 
…ls into mint-tax-and-cash
@Locrian24
prerelease 1.8.0-140
26c8f46
@zachharrison
Fix css part bug in share button
19079dc
@00salmon
match order of props in component to template, fixed a couple small i…
57302f9 
…ssues
@00salmon
Update sqm-tax-and-cash.tsx
66864d7
@Ultha258
Updated the spec to include if same tax doc was found in impact vs Ad…
50e3ce3 
…vocate
@Locrian24
fix states for preexisting users with mismatched document types
3313aed
@Locrian24
Merge branches 'mint-tax-and-cash' and 'mint-tax-and-cash' of github.…
f5d9685 
…com:saasquatch/program-tools into mint-tax-and-cash
@Locrian24
prerelease 1.8.0-141
c0e9eaf
@00salmon
update changelog release date
77820e9
@00salmon
prep PR for launch
5c3886d
@Locrian24
fix logic bug with mismatched tax documents in docusign initial state
b0cd905
@Locrian24
Merge branch 'mint-tax-and-cash' of github.com:saasquatch/program-too…
f8ac9d2 
…ls into mint-tax-and-cash
@Locrian24
prerelease 1.8.0-142
46bc4f1
@00salmon 00salmon requested a review from Locrian24 April 10, 2024 17:03
@00salmon 00salmon merged commit ce4bf0a into master Apr 10, 2024
9 of 12 checks passed
@00salmon 00salmon deleted the mint-tax-and-cash branch April 10, 2024 22:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@00salmon 00salmon 00salmon left review comments

@AndiLuo AndiLuo AndiLuo left review comments

@DerekSiemens DerekSiemens DerekSiemens left review comments

@zachharrison zachharrison zachharrison left review comments

@Locrian24 Locrian24 Locrian24 approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@noahwc @00salmon @AndiLuo @Locrian24 @DerekSiemens @zachharrison @Ultha258