NPM Resource Changed PBF ? ? ? Pending

User tests: Successful: Unsuccessful:

avatar niharikamahajan02
niharikamahajan02
7 Jan 2022

Pull Request for Issue #35714 .

Summary of Changes

Removed "temporary" css rules as mentioned in issue.

avatar niharikamahajan02 niharikamahajan02 - open - 7 Jan 2022
avatar niharikamahajan02 niharikamahajan02 - change - 7 Jan 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 7 Jan 2022
Category Administration Templates (admin) NPM Change Front End Templates (site)
avatar richard67
richard67 - comment - 7 Jan 2022

@niharikamahajan02 Could you give this pull request a more meaningful title? Maybe something like

[4.0] Remove "(TEMPORARY)" css rules in main cassiopea template.scss

And maybe provide some testing instructions, too.

Thanks in advance.

avatar niharikamahajan02 niharikamahajan02 - change - 7 Jan 2022
Title
changes done
[4.0] Remove "(TEMPORARY)" css rules in main cassiopea template.scss
avatar niharikamahajan02 niharikamahajan02 - edited - 7 Jan 2022
avatar niharikamahajan02 niharikamahajan02 - change - 7 Jan 2022
The description was changed
avatar niharikamahajan02 niharikamahajan02 - edited - 7 Jan 2022
avatar niharikamahajan02
niharikamahajan02 - comment - 7 Jan 2022

Done with the required changes.

avatar Quy Quy - change - 17 Jan 2022
Labels Added: ? NPM Resource Changed ?
avatar Quy
Quy - comment - 17 Jan 2022

I have tested this item successfully on cf42afa


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36596.

avatar Quy Quy - test_item - 17 Jan 2022 - Tested successfully
avatar Quy
Quy - comment - 29 Jan 2022

Please fix conflicts.

avatar niharikamahajan02
niharikamahajan02 - comment - 30 Jan 2022

Please fix conflicts.

I am not able to know what the conflict is in these 2 files .
administrator/templates/atum/scss/blocks/_global.scss
templates/cassiopeia/scss/template.scss

May I know these conflicts please.

avatar richard67
richard67 - comment - 30 Jan 2022

I am not able to know what the conflict is in these 2 files . administrator/templates/atum/scss/blocks/_global.scss templates/cassiopeia/scss/template.scss

May I know these conflicts please.

@niharikamahajan02 In general you will not see any conflicts when just checking your files or when just checking the changes on GitHub.

Conflicts (which means merge conflicts, not political conflicts ? ) will appear when you have made changes in some file, and in the base branch of your branch other changes have been made in the same file at the same place.

Here we had to change the base branch of your PR from 4.0-dev to 4.1-dev because 4.0 has end of development, and 4.1 is the current version soon, so development (like bug fixes) continues on that version and so on the 4.1-dev branch, and new features will go into 4.2.

With version 4.1 the 2 files which have the conflicts have been moved from the template to the media folder because the templates have been changed in 4.1 to support child templates, and that's the conflict now:

  • "administrator/templates/atum/scss/blocks/_global.scss" has been moved to "build/media_source/templates/administrator/atum/scss/blocks/_global.scss"
  • "templates/cassiopeia/scss/template.scss" has been moved to "build/media_source/templates/site/cassiopeia/scss/template.scss"

(When the scss files are compiled the results will be not in "build/media_source" but in the same place below "media", but this is just for your understanding, it is not relevant for solving the conflict.)

If a conflict is easy, there is a button on GitHub so solve it there in an editor. But in this case the conflict seems to be too complicated for the GitHub web site, so the button is disabled.

So these conflicts have to be solved in your local git clone with help of the git command line (normally a CMD window on Windows and a Bash shell window on Linux).

The conflicts are then solved by merging the 4.1-dev branch of the upstream repository (this one here) into your branch for this PR. Git will then tell you that there are conflicts which you have to solve. In this case here you solve them by applying the changes which have made here in the files at the old place to the files at the new place and accept that the old files are deleted.

You can find some documentation on how to solve conflicts here: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/addressing-merge-conflicts/resolving-a-merge-conflict-using-the-command-line

If you want to learn that - and I think you should because being familiar with come code management and version control system like Git is a key skill for people who develop software in companies, teams, as soon as more than 1 person works on a software - then try to solve it yourself,

If you run into problems and it does not work for you, either call here for help, or (maybe the easiest way if you messed this branch with trying to solve conflicts) you make a new PR for the 4.1-dev branch which makes the changes from here but at the files at their new locations.

avatar niharikamahajan02 niharikamahajan02 - change - 30 Jan 2022
Labels Added: Conflicting Files ?
Removed: ?
avatar joomla-cms-bot joomla-cms-bot - change - 30 Jan 2022
Category Administration Templates (admin) NPM Change Front End Templates (site) Repository NPM Change
avatar niharikamahajan02
niharikamahajan02 - comment - 30 Jan 2022

@richard67 sir thanks for telling . Are the commits that I made now correct?

avatar niharikamahajan02
niharikamahajan02 - comment - 30 Jan 2022

@richard67 sir this concept is new to me , please tell if the commit i did is correct , thanks.

avatar richard67
richard67 - comment - 30 Jan 2022

@niharikamahajan02 I see you have created a new PR #36897 . Does that PR replace this one here? If so, why haven’t you closed this one here. Do you really think it helps us if people keep their duplicate PRs open so we have to identify them and close the duplicates?

avatar niharikamahajan02
niharikamahajan02 - comment - 31 Jan 2022

ok @richard67 sir , I've closed that PR , this is the final PR .Thanks

avatar niharikamahajan02 niharikamahajan02 - change - 18 Apr 2022
Labels Added: PBF
Removed: Conflicting Files ?
avatar HLeithner
HLeithner - comment - 27 Jun 2022

This pull request has automatically rebased to 4.2-dev.

avatar joomla-bot
joomla-bot - comment - 27 Jun 2022

This pull requests has been automatically converted to the PSR-12 coding standard.

avatar HLeithner HLeithner - change - 27 Jun 2022
Labels Added: ?
avatar brianteeman
brianteeman - comment - 9 Sep 2022

It is not possible any more to see why the drone build for this pr was failing but perhaps it is because the tests need to be updated to remove this test

* Selects an option in a Chosen Selector based on its id.

avatar drmenzelit drmenzelit - change - 22 Oct 2022
Labels Added: ? ?
Removed: ?
avatar bembelimen bembelimen - change - 22 Oct 2022
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-10-22 14:18:26
Closed_By bembelimen
avatar bembelimen bembelimen - close - 22 Oct 2022
avatar bembelimen bembelimen - merge - 22 Oct 2022
avatar bembelimen
bembelimen - comment - 22 Oct 2022

Thx

Add a Comment

Login with GitHub to post a comment