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

MDL-77991 - [Docs] Add documentation for how to use the tertiary search #627

Merged
merged 2 commits into from
Apr 17, 2024

Conversation

Chocolate-lightning
Copy link
Collaborator

No description provided.

@netlify
Copy link

netlify bot commented May 18, 2023

Deploy Preview for moodledevdocs ready!

Name Link
🔨 Latest commit e7e5ac6
🔍 Latest deploy log https://app.netlify.com/sites/moodledevdocs/deploys/661fbe23c5a54c000861a6a6
😎 Deploy Preview https://deploy-preview-627--moodledevdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions
Copy link
Contributor

github-actions bot commented May 18, 2023

⚡️ Lighthouse report for the deploy preview of this PR

URL Performance Accessibility Best Practices SEO PWA Report
/ 🔴 37 🟢 95 🟢 100 🟢 90 🟢 100 Report
/docs/4.4/apis/commonfiles 🟠 64 🟢 93 🟢 100 🟢 100 🟢 100 Report
/general/development/gettingstarted 🟠 77 🟢 95 🟢 100 🟢 90 🟢 100 Report
/general/releases 🟠 68 🟢 95 🟢 100 🟢 100 🟢 100 Report

Copy link
Member

@andrewnicols andrewnicols left a comment

Choose a reason for hiding this comment

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

Just a couple of minor changes to do with coding style. Looks good to me.

It would be good to also document this in the devupdate.md for 4.3 (https://github.com/moodle/devdocs/blob/main/docs/devupdate.md).

@Chocolate-lightning
Copy link
Collaborator Author

Kia ora @andrewnicols,

Thanks for checking this out. I am glad it was only some obvious code styling issues to amend. I'll squash down the created commits from accepting your suggestions and I'll look at updating devupdate.md then I'll get back to you.

Cheers,

@sarjona sarjona changed the title [Docs] Add documentation for how to use the tertiary search [Docs] Add documentation for how to use the tertiary search - MDL-77991 Jun 7, 2023
@sarjona sarjona added the blocked-by-tracker This issue is blocked by an issue in https://tracker.moodle.org/ label Jun 7, 2023
@sarjona sarjona changed the title [Docs] Add documentation for how to use the tertiary search - MDL-77991 MDL-77991 - [Docs] Add documentation for how to use the tertiary search Jun 7, 2023
@sarjona sarjona removed the blocked-by-tracker This issue is blocked by an issue in https://tracker.moodle.org/ label Aug 11, 2023
@sarjona
Copy link
Member

sarjona commented Aug 11, 2023

I've removed the tag "blocker-by-tracker" because this issue was integrated a few weeks ago.

@Chocolate-lightning could you please rebase your branch? TIA! :-)

@sarjona
Copy link
Member

sarjona commented Dec 1, 2023

@Chocolate-lightning is there any chance you can rebase your branch to the latest? TIA! :-)

@sarjona sarjona force-pushed the tertiarysearching branch from 3885faf to 944b12f Compare April 17, 2024 11:55
@sarjona sarjona force-pushed the tertiarysearching branch from 944b12f to e7e5ac6 Compare April 17, 2024 12:18
Copy link
Member

@sarjona sarjona left a comment

Choose a reason for hiding this comment

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

Hi @Chocolate-lightning!
Thanks for adding this documentation and taking into account Andrew's suggestions.
I've amended your commit adding the missing files (versioned_docs/version-4.3/guides/javascript/comboboxsearch/examples.md and versioned_docs/version-4.3/guides/javascript/comboboxsearch/index.md) to fix the errors raised by GHA checks ;-)

@sarjona sarjona merged commit 0188b9b into moodle:main Apr 17, 2024
7 checks passed
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