Skip to content
This repository has been archived by the owner on Jun 27, 2024. It is now read-only.

refactor(ext/files): respect index.html and 404.html files #197

Merged
merged 3 commits into from
Aug 24, 2023
Merged

Conversation

not-ivy
Copy link
Contributor

@not-ivy not-ivy commented Aug 23, 2023

feat: files ext now look for index.html if no file is specified

feat: files ext now look for 404.html if file could not be found

feat: files ext now look for index.html if no file is specified

feat: files ext now look for 404.html if file could not be found
@boywithkeyboard
Copy link
Collaborator

@not-ivy i'd suggest to split this pr into 3 smaller ones as it's better to have one (major) change per commit.

@boywithkeyboard
Copy link
Collaborator

...or at least 2, one for the s3 functionality and the other one for the html files lookup

@not-ivy not-ivy marked this pull request as ready for review August 23, 2023 19:21
ext/files.ts Outdated Show resolved Hide resolved
@boywithkeyboard boywithkeyboard changed the title refactor: files extention refactor(ext/files): respect index.html and 404.html files Aug 23, 2023
Copy link
Collaborator

@boywithkeyboard boywithkeyboard left a comment

Choose a reason for hiding this comment

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

lgtm

@boywithkeyboard boywithkeyboard merged commit 2f886c8 into boywithkeyboard-archive:dev Aug 24, 2023
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants