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

[ticket/16913] Add Search Index Progress Bar with Stats [3.3.x] #6410

Merged
merged 2 commits into from Jan 31, 2023

Conversation

Dark1z
Copy link
Contributor

@Dark1z Dark1z commented Jul 27, 2022

PR for phpbb:master is at #6441 .


ScreenShot:
.
1st run (On Submit) :
search_1st_run
.
2nd & consecutive run (On each Refresh) :
search_nth_run
.
Continue to run (In-case the Indexing was Interrupted) :
search_cont_run
.
Changes :

  • Update SQL for count of posts
  • Update search index test
  • Removed "Pop-up Progress Bar".
  • For first run display the "Success Message".
  • Display the Info from "Pop-up Progress Bar" to "Success Message".
  • Only display "Redirect" & "Rate" of post refreshed "Success Message".
  • Minor Language Fix.
  • Use ORDER BY post_id ASC for progress stats.
  • Removed HTML from Lang String.
  • Moved HTML to PHP file.
  • Increased the size of Progress-Bar by 2x.

Checklist:

  • Correct branch: master for new features; 3.3.x for fixes
  • Tests pass
  • Code follows coding guidelines: master and 3.3.x
  • Commit follows commit message format

Tracker ticket (set the ticket ID to your ticket ID):

https://tracker.phpbb.com/browse/PHPBB3-16913

@marc1706
Copy link
Member

I have played around a bit with your pull request. Generally we're trying to move away from using HTML in PHP files which is also why I started adjusting this a bit: DaRk1V0iD/phpbb@ticket/16913_3.3.x...marc1706:phpbb:ticket/16913_3.3.x

After my changes, the progress bar looks as follows:
image
image

IMHO the alignment of the progress bar when being asked about continuing the indexing also fits in better now. If you want I can also add my commit on top of your changes here.

@Dark1z
Copy link
Contributor Author

Dark1z commented Nov 14, 2022

I have played around a bit with your pull request. Generally we're trying to move away from using HTML in PHP files which is also why I started adjusting this a bit: DaRk1V0iD/phpbb@ticket/16913_3.3.x...marc1706:phpbb:ticket/16913_3.3.x
...
IMHO the alignment of the progress bar when being asked about continuing the indexing also fits in better now. If you want I can also add my commit on top of your changes here.

Yes! please do any required changes which you think is most appropriate.

My aim was that the end user is well informed about the current (to be more precise last) completed status of search index,
So that users can estimate by assumption or inference of how long it might take to finish the search index.

Best regards 👍

@marc1706 marc1706 added this to the 3.3.10 milestone Nov 14, 2022
@marc1706
Copy link
Member

Btw, if you enable to allow edits to this PR, I could directly add my changes (should be on the right side below the participants info):
image
If you allow the same for the master version of this PR I can also update that.

@Dark1z
Copy link
Contributor Author

Dark1z commented Nov 15, 2022

Btw, if you enable to allow edits to this PR, I could directly add my changes (should be on the right side below the participants info): image If you allow the same for the master version of this PR I can also update that.

I can not see that option:
image
Due to this issue https://github.com/orgs/community/discussions/5634

Instead can I give you maintainer role in that repository ?


Edit:
Or should I take the pull from here: DaRk1V0iD/phpbb@ticket/16913_3.3.x...marc1706:phpbb:ticket/16913_3.3.x ?


Best regards 👍

@marc1706
Copy link
Member

Yes, you should be able to pull from my branch via fast forward and then push to yours.

Dark1z and others added 2 commits November 17, 2022 02:20
- Update SQL for count of posts
- Update search index test
- Removed "Pop-up Progress Bar".
- For first run display the "Success Message".
- Display the Info from "Pop-up Progress Bar" to "Success Message".
- Only display "Redirect" & "Rate" of post refreshed "Success Message".
- Minor Language Fix.
- Use `ORDER BY post_id ASC` for progress stats.
- Removed HTML from Lang String.
- Moved HTML to PHP file.
- Increased the size of Progress-Bar by 2x.

PHPBB3-16913
@Dark1z
Copy link
Contributor Author

Dark1z commented Nov 16, 2022

Yes, you should be able to pull from my branch via fast forward and then push to yours.

Done.

@marc1706 marc1706 modified the milestones: 3.3.10, 3.3.11 Jan 9, 2023
@marc1706 marc1706 merged commit 1b82ea8 into phpbb:3.3.x Jan 31, 2023
@marc1706
Copy link
Member

And merged. Changes will be included in 3.3.11. Thanks for your contribution. 🚀

@Dark1z Dark1z deleted the ticket/16913_3.3.x branch January 31, 2023 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants