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

speed up entries display by no longer looking at all the status objects #988

Merged
merged 5 commits into from
Dec 5, 2024

Conversation

orangewolf
Copy link
Member

@orangewolf orangewolf commented Dec 3, 2024

we lose the error message in the title, you have to click through to see it but page speeds go up 100x.

Release note:
Requires running rails bulkrax:update_status_messages for error messages to display.

@orangewolf orangewolf added the minor-ver for release notes label Dec 3, 2024
@bkiahstroud bkiahstroud added major-ver for release notes and removed minor-ver for release notes labels Dec 5, 2024
@orangewolf orangewolf merged commit 842a9e8 into main Dec 5, 2024
8 checks passed
@orangewolf orangewolf deleted the faster_entries_list branch December 5, 2024 23:22
sephirothkod pushed a commit that referenced this pull request Dec 17, 2024
…ts (#988)

* speed up entries display by no longer looking at all the statusable objects

* update importer an exporter too

* fixing importers/exporters

* migration version fix

* improve initial importer show page load time
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
major-ver for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants