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

#26 integers disabled samecm nocm #27

Merged
merged 8 commits into from
Feb 26, 2024

Conversation

danowar2k
Copy link
Contributor

This PR fixes all of the problems identified in #26:

  • the current course module is not offered for selection for the relative course module
  • options where the module completion is disabled have the disabled attribute placement fixed
  • always have an explicit course module initially selected as default to prevent "none" selection
  • store JSON values as integers, not strings
  • do not store the course module id if the relative date mode is not "relative to module completion"

@danowar2k
Copy link
Contributor Author

Fixed the Moodle nagging (I stand by my opinion that having everything lower case makes reading vars worse).

Also (probably, still running) fixed the behat test. I used the wrong member of the value variable (got confused because of one letter variables).

Maybe add readable constants to the YUI module?

const RELATIVE_START = 's';

etc.

@danowar2k
Copy link
Contributor Author

danowar2k commented Feb 21, 2024

I'm stumped with the new error. I currently haven't fully setup my local behat, and the actions output doesn't show the full page HTML. When I do the stuff locally in my dev environment, the behat steps would succeed. This seems very strange.

--- Failed steps:

001 Scenario: Add relative condition ui with a module               # /home/runner/work/moodle-availability_relativedate/moodle-availability_relativedate/moodle/availability/condition/relativedate/tests/behat/availability_relativedate_ui.feature:63
      Then I should see "1 day after completion of activity Page A" # /home/runner/work/moodle-availability_relativedate/moodle-availability_relativedate/moodle/availability/condition/relativedate/tests/behat/availability_relativedate_ui.feature:73
        "1 day after completion of activity Page A" text was not found in the page

EDIT: Explanation for the missing Selenium: My current dev environment is, sadly, Solaris. I'm transitioning to RHEL, so I avoid getting anything to work on Solaris if I don't really have to. Economics of time and sanity.

@danowar2k
Copy link
Contributor Author

Better wait for my Github action to complete: https://github.com/danowar2k/moodle-availability_relativedate/actions/runs/8038749498

@danowar2k
Copy link
Contributor Author

Finally. It lives!!!!

@ewallah ewallah merged commit 520f33e into ewallah:main Feb 26, 2024
7 of 8 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.

2 participants