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

[5.3] Remove autostart temp code #44287

Merged
merged 1 commit into from
Oct 17, 2024
Merged

Conversation

brianteeman
Copy link
Contributor

#43031 introduced some code

At this point, the autostart is not present as an option to the user. Once we have the full functionality (we will make auto-start as an option to the tour in the form), we can remove that additional test.

The full functionality is now available and this code can be removed.

Testing is simple just try and duplicate an existing tour. It should be successfully duplicated both with and without this PR

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

joomla#43031 introduced some code

> At this point, the autostart is not present as an option to the user. Once we have the full functionality (we will make auto-start as an option to the tour in the form), we can remove that additional test.

The full functionality is now available and this code can be removed.

Testing is simple just try and duplicate an existing tour. It should be successfuly duplicated both with and without this PR

Signed-off-by: BrianTeeman <brian@teeman.net>
@rdeutz rdeutz merged commit 20868e5 into joomla:5.3-dev Oct 17, 2024
3 checks passed
@rdeutz rdeutz added this to the Joomla! 5.3.0 milestone Oct 17, 2024
@brianteeman
Copy link
Contributor Author

Thanks

@brianteeman brianteeman deleted the autostart branch October 17, 2024 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants