Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@rehanrehman389
Copy link
Contributor

No description provided.

@rehanrehman389
Copy link
Contributor Author

@pateljannat Hi, I’m assuming all the resumes that were uploaded are public.

@cypress
Copy link

cypress bot commented Oct 29, 2025

lms    Run #1691

Run Properties:  status check passed Passed #1691  •  git commit 9408c86ac4 ℹ️: Merge 9c38444c4b24c07d87797a85cc27bec54e44fb72 into 6a64048bb6df7184cea65b75c764...
Project lms
Branch Review fix-resume-url
Run status status check passed Passed #1691
Run duration 00m 59s
Commit git commit 9408c86ac4 ℹ️: Merge 9c38444c4b24c07d87797a85cc27bec54e44fb72 into 6a64048bb6df7184cea65b75c764...
Committer Rehan Ansari
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 2
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.
View all changes introduced in this branch ↗︎

@pateljannat
Copy link
Collaborator

@rehanrehman389 make them private through the same patch, and also make sure they are linked to the LMS Job Application Document. Also, the document should then be shared with the job poster so they have access to the resume

@rehanrehman389
Copy link
Contributor Author

rehanrehman389 commented Nov 1, 2025

Also, the document should then be shared with the job poster so they have access to the resume

Currently, the LMS Student role has read access to the LMS Job Application, so everyone with this role can access the resume.
So, I changed the "Only If Creator" checkbox.

@pateljannat pateljannat merged commit 66c2ec0 into frappe:develop Nov 7, 2025
6 checks passed
@frappe-pr-bot
Copy link
Collaborator

🎉 This PR is included in version 2.40.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants