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 delivery #268

Merged
merged 71 commits into from
Sep 2, 2024
Merged

Refactor delivery #268

merged 71 commits into from
Sep 2, 2024

Conversation

tschumpr
Copy link
Contributor

@tschumpr tschumpr commented Aug 22, 2024 •
edited
Loading

Resolves #246

Alter Code wird mit #262 aufgeräumt
FetchApi wird mit #271 nochmals angeschaut

@tschumpr
Use GlobalStyles to override default
0db0fca
@tschumpr
Move markdownContent to components
0373935
@tschumpr
Update default font size
95efa8b
@tschumpr
Added styled components for flex layout
200acfb
@tschumpr
Expand styled flex box component
9e6f09a
@tschumpr
Add file dropzone component
4b6f068
@tschumpr
Fix translation
1c7bcbd
@tschumpr
Add loading spinner to stepper icon
ad8dd26
@tschumpr
Add loading spinner to stepper icon
35b7a08
@tschumpr
Support in-page navigation in about.tsx
19b9faf
@tschumpr
Simplify code
d5e88fa
@tschumpr
Move button styling to theme
ee6b800
@tschumpr
Remove unused code
bb0449d
@tschumpr
Add form components
ed4def3
@tschumpr
Add disabled prop to file dropzone
78b3af9
@tschumpr
Update styled flex components
e2fdeb7
@tschumpr
Add delivery flow
ae5f7c2
@tschumpr
Show validation error status in step label
3b3b0b7
@tschumpr
Fix step label font weight
9a886d8
@tschumpr
Return full validation response
bfc4096
@tschumpr
Fix canceling requests
5fe628c
@tschumpr
Fix cancellation during validation
02b183e
@tschumpr
Fix warning
b191b00
@tschumpr
Update primary color
1b7dfb4
@tschumpr
Support label addition and step errors
d68bb7d
@tschumpr
Only show cancellation button while active
d7d0481
@tschumpr
Allow cancel when not logged in
15587ac
@tschumpr
Show file error in step title
cb2b333
@tschumpr
Fix dropzone height
4e8efa5
@tschumpr
Add initialized check to appSettings
308356e
tschumpr and others added 25 commits August 29, 2024 11:29
@tschumpr
Add delivery tests
606c8da
@tschumpr
Reset submit error on retry
9472517
@tschumpr
Support both text and json error responses
facea44
@tschumpr
Add missing response
9799448
@tschumpr
Update package-lock.json
a34d220
@tschumpr
Update package-lock.json
1c93eb0
@tschumpr
Fix logout button name
ef126b9
@tschumpr
Mock termsOfUse
7247dd0
@tschumpr
Mock validation responses
3c6f7a0
@tschumpr
Don't use mock in complete delivery process
ef2c540
@tschumpr
Don't check isLoading for upload when using API
8f81b31 
API calls are too fast so we can only check isLoading when we use the
mocked calls.
@tschumpr
Remove submit loading check
3a45d21 
API is too fast
@tschumpr
Mock all api calls
68f1e3a
@tschumpr
Wait max duration
ac13b4e
@tschumpr
Increase maximum duration
f278603
@tschumpr
Revert changes
ed24a3c
@tschumpr
Update volume mapping
1a2ee7b
@tschumpr
Set wait timeout
dc5be11
@tschumpr
Increase timeout
55f9e76
@tschumpr
Remove wait for debugging
8efc910
@tschumpr
Re-add wait
53fd9b1
@tschumpr
Span delivery box over full height
bdc249f
@tschumpr
Merge branch 'rebranding' into refactor-delivery
ea364b6
@tschumpr
Remove timeout
463f154
@tschumpr
Add delay to show submit loading spinner
983516a
@tschumpr tschumpr marked this pull request as ready for review September 2, 2024 09:07
Philippluca
Philippluca approved these changes Sep 2, 2024
View reviewed changes
Copy link
Member

@Philippluca Philippluca left a comment

Choose a reason for hiding this comment

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

LGTM

src/Geopilot.Frontend/src/components/buttons.tsx Show resolved Hide resolved
@tschumpr tschumpr merged commit a3453a8 into rebranding Sep 2, 2024
2 checks passed
@tschumpr tschumpr deleted the refactor-delivery branch September 2, 2024 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Philippluca Philippluca Philippluca 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.
2 participants
@tschumpr @Philippluca