Comments (4)
hinakhadim
commented on August 17, 2024
1
Further debugging leads me to this conclusion:
Here, the variable is setting the first time value of LMS_BASE_URL because till then mergeConfig hasn't been run.
For further reading, This link can be helpful. Link
I believe making courseUneroll a function instead of a variable will fix the issue, so that courseUnenroll url picks the latest/current value of LMS_BASE_URL. If this seems good, I am willing to work on it.
from frontend-app-learner-dashboard.
justinhynes
commented on August 17, 2024
Thanks for your report! The Aperture team will be tracking this internally as ticket APER-3566.
from frontend-app-learner-dashboard.
jsnwesson
commented on August 17, 2024
Hi @hinakhadim , the Aperture and I talked about this issue during our meeting, and we're okay with you continuing to pursue a fix for this if you have the time. I think your approach of converting the variable into a function to fetch the appropriate URL is a viable option. Let us know if you have any questions!
from frontend-app-learner-dashboard.
hinakhadim
commented on August 17, 2024
Hey everyone,
Thanks for your time. I think that this issue should close automatically after merging this PR.
from frontend-app-learner-dashboard.
Related Issues (20)
- Missing internationalization HOT 4
- Filter Form header (Course Status) is not marked for translation
- Overhangio/openedx-learner-dashboard-dev, repository does not exist?
- [Console Warning]: Cannot update a component (`DashboardLayout`) while rendering a different component (`WidgetSidebar`).
- Make Upgrade button optional in Learner Dashboard HOT 3
- Node 20 upgrade
- Not able to un-enroll HOT 3
- Dependency Dashboard
- Use configured platform name in email reminder, not "edX" HOT 4
- Includes ten separate copies of the Paragon SCSS
- Reimplement "Looking for a new challenge" panel HOT 1
- Create components for `PluginSlots`, standardize id naming
- Extract RecommendationsPanel code from the MFE
- Don't show Programs button if programs are not configured HOT 1
- Don't show help button if no support page is configured HOT 1
- Widgets and WidgetContainers vs frontend-plugin-framework PluginSlots
- [DEPR]: Optimizely support HOT 1
- [DEPR]: RecommendationsPanel
- Error in the Courses link and in the Discover link HOT 1
- Test frontend-app-learner-dashboard on Ubuntu 24.04
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 frontend-app-learner-dashboard.