Comments (1)
Issue Description:
After further investigation of this issue, I think I've found where the conflicts might be happening. If a user were to change the html of an issue prior to marking it as resolved, then UDOIT will not be able to find that HTML upon reloading the state, hence the "Fixed HTML was not replaced in LMS" error. Important to note however, is that every time I fixed an error manually and marked it as resolved, that error was not found after a refresh of the UDOIT LTI (but it was also not found in the resolved issues filter as well. I think the two fixes to this issue would be:
1.) Find a less generic and a less intimidating error message than "Fixed HTML was not found in LMS. Please contact administrator" so as to let the user know that their changes will actually be reflected in both UDOIT and the LMS.
2.) Find a way to retain the previous state of the issue in order to show it in the resolved issues section.
Images:
Before manually fixing the issue:
The issue after the user checks "mark as resolved" after the html content has been altered:
Searching with the "resolved issues" filter (as you can see, the issue in question does not show up in as a manually resolved issue):
Number of issues after the user manually fixes the issue:
from udoit.
Related Issues (20)
- Transparent background-color styling throwing "Insufficient Text Color Contrast" error HOT 1
- Replace file references on file replacement
- Switch UDOIT to use the ucfopen/phpally repo
- Live Reload Update
- High False Positive Rate with Avoid Using Styles for Document Structure HOT 1
- Update GitHub Actions to reflect latest supported version of PHP
- Packages do not support PHP 8.3
- Update or remove old Dockerfile
- UFIXIT recognizes problems with HTML file and offers solutions that cannot be applied HOT 1
- Update all IMS Global Domain URLs from `http` to `https` HOT 1
- UDOIT checks for only a few phrases and words to mark a link as suspicious.
- Rework the modal message for Updating Filetypes
- Mark issues as reviewed when a user replaces a file in 'Review Files'
- UDOIT returns a 413 error whenever a file larger than 1MB is uploaded
- Remove display of data in browser console HOT 1
- Create a Makefile
- Add Python Support For Potential Machine Learning Integrations
- VideoApiVimeoConnectionFailed for all Vimeo links
- Empty Tables issue currently has no option to resolve it through Canvas
- Add a changelog file to keep track of ongoing updates
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from udoit.