Skip to content

Cleanup dependencies a bit more #933

Merged
fliiiix merged 8 commits intomainfrom
feature/web-cleanup
Apr 10, 2026
Merged

Cleanup dependencies a bit more #933
fliiiix merged 8 commits intomainfrom
feature/web-cleanup

Conversation

@fliiiix
Copy link
Copy Markdown
Member

@fliiiix fliiiix commented Apr 2, 2026

long term i think it would be better to not have any of this js crap 🙈

@fliiiix fliiiix requested review from randombenj and reglim April 2, 2026 11:00
Comment thread web/src/components/IFrame.tsx Outdated
Comment thread web/src/components/InfoBanner.tsx Outdated
@fliiiix fliiiix force-pushed the feature/web-cleanup branch from b492c12 to 124c133 Compare April 7, 2026 06:22
Copy link
Copy Markdown
Member

@randombenj randombenj left a comment

Choose a reason for hiding this comment

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

Nice, yeah agreed that we should keep drpendencies as few as possible

@fliiiix
Copy link
Copy Markdown
Member Author

fliiiix commented Apr 9, 2026

@ioasjfopas i picked now the solution from @tbuerli-komax and this seems to persist the /latest if you both could give this a quick review if something is still not working would be great

i testest a few things and didn't notice any issues with the current version

@tbuerli-komax
Copy link
Copy Markdown
Contributor

From my point of view, this fixes the browser-back trough latest navigation.

Other than that, I just saw that the following two files can be removed as the only reference to them gets removed in the last commit:

  • web/src/pages/IframePageNotFound.tsx
  • web/src/style/pages/IframePageNotFound.module.css

@fliiiix fliiiix force-pushed the feature/web-cleanup branch from fe1679a to d47ffbc Compare April 9, 2026 14:51
@fliiiix
Copy link
Copy Markdown
Member Author

fliiiix commented Apr 9, 2026

good catch removed the unused files

@fliiiix fliiiix force-pushed the feature/web-cleanup branch from d47ffbc to 0cb9c37 Compare April 10, 2026 06:54
@fliiiix
Copy link
Copy Markdown
Member Author

fliiiix commented Apr 10, 2026

thanks @tbuerli-komax for the navigation fix

@fliiiix fliiiix merged commit 04ca027 into main Apr 10, 2026
8 checks passed
@fliiiix fliiiix deleted the feature/web-cleanup branch April 10, 2026 07:43
@fliiiix fliiiix mentioned this pull request Apr 10, 2026
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.

3 participants