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

[DEV-1517]: questions page reactiveness #759

Merged
merged 13 commits into from
Apr 4, 2024

Conversation

MikeAtUqido
Copy link
Collaborator

List of Changes

  • Update the local state of the questions table to increase overall reactiveness

Motivation and Context

The users who made an action on the questions page had to wait the refresh delay to see the action

How Has This Been Tested?

Manually

Screenshots (if appropriate):

Types of changes

  • Chore (nothing changes by a user perspective)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

Copy link

changeset-bot bot commented Mar 19, 2024

🦋 Changeset detected

Latest commit: 6c1d1d3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
nextjs-website Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

apps/nextjs-website/src/config.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@datalek datalek left a comment

Choose a reason for hiding this comment

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

⚠️ This approval is about the config.ts file only, I did not checked the React part

Copy link
Contributor

github-actions bot commented Apr 4, 2024

Jira Pull Request Link

This Pull Request refers to the following Jira issue DEV-1517

Copy link
Contributor

github-actions bot commented Apr 4, 2024

📦 Next.js Bundle Analysis for nextjs-website

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@MarcoPonchia MarcoPonchia merged commit 5852a9a into main Apr 4, 2024
11 checks passed
@MarcoPonchia MarcoPonchia deleted the feature/DEV-1517-questions-page-reactiveness branch April 4, 2024 15:09
MikeAtUqido added a commit that referenced this pull request Apr 5, 2024
* feat(DEV-1517): more reactiveness on questions page

* chore: add changeset

* feat(DEV-1517): fix PR comments

* feat(DEV-1517): increase refresh interval

* fix(DEV-1517): fix PR comments

---------

Co-authored-by: Marco Ponchia <[email protected]>
marcobottaro added a commit that referenced this pull request Apr 9, 2024
* feat(DEV-1566): rename textContent field

* chore: add changeset

* chore: remove code

* chore: update changeset

Co-authored-by: Marco Ponchia <[email protected]>

* [DEV-1586] Add GOOGLE_OAUTH_REDIRECT_URI env (#813)

* [DEV-1517]: questions page reactiveness (#759)

* feat(DEV-1517): more reactiveness on questions page

* chore: add changeset

* feat(DEV-1517): fix PR comments

* feat(DEV-1517): increase refresh interval

* fix(DEV-1517): fix PR comments

---------

Co-authored-by: Marco Ponchia <[email protected]>

* Update CODEOWNERS (#807)

* [DPC-128] Add new tutorials for pagoPA (#809)

* chore: update changeset

* chore: remove unused files

---------

Co-authored-by: Marco Ponchia <[email protected]>
Co-authored-by: Jeremy Gordillo <[email protected]>
Co-authored-by: Marco Comi <[email protected]>
Co-authored-by: Monica Costantini <[email protected]>
Co-authored-by: marcobottaro <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants