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.

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

v3.2.5 [Frontend Module] #127

Merged
merged 4 commits into from
May 11, 2024
Merged

v3.2.5 [Frontend Module] #127

merged 4 commits into from
May 11, 2024

Conversation

Signed-off-by: Dicky Herlambang (花) <herlambangdicky5@gmail.com>
FBE-008: Include more advanced features for HTMLtoPDF
Background:
- As the user, i want more to modified or customized PDF that i was convert from HTML or website before. Because current feature are less customizable and just convert it like normal browser

Possibility Issue:
- Document positioning or layout can be mess if webpage are not optimized for single page layout

Resolution:
- Implement new features and toggle into HTMLtoPDF page for several customization from iLovePDF API

Referecences:
- https://developer.ilovepdf.com/docs/guides/html-pdf-api

FE-010: Fix multiple same API request

Background:
For some reason, api/v1/logs/limit has call it twice due unproper async logic. This can make BE service treat that user has 2 request in same time and causing rate limiting hit the limit fastest than it should be.

Possibility Issue:
- Rate limiting from BE can be hit in a time if user do refresh the page accidentally

Resolution:
- Wrap async function after button click to avoid async logic run twice before and after button click

Signed-off-by: Dicky Herlambang (花) <herlambangdicky5@gmail.com>
Signed-off-by: Dicky Herlambang (花) <herlambangdicky5@gmail.com>
Signed-off-by: Dicky Herlambang (花) <herlambangdicky5@gmail.com>
@Nicklas373 Nicklas373 added bug Something isn't working enhancement New feature or request dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code security php Pull requests that update Php code labels May 11, 2024
@Nicklas373 Nicklas373 self-assigned this May 11, 2024
public/build/assets/kao-logic-DwIvi7le.js Dismissed Show dismissed Hide dismissed
public/build/assets/kao-logic-DwIvi7le.js Dismissed Show dismissed Hide dismissed
public/build/assets/kao-logic-DwIvi7le.js Dismissed Show dismissed Hide dismissed
public/build/assets/kao-logic-DwIvi7le.js Dismissed Show dismissed Hide dismissed
@Nicklas373 Nicklas373 merged commit 8358413 into fe/master May 11, 2024
2 checks passed
@Nicklas373 Nicklas373 deleted the dev/fe/master-live branch May 11, 2024 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dependencies Pull requests that update a dependency file enhancement New feature or request javascript Pull requests that update Javascript code php Pull requests that update Php code security
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant