Informations

[chack1172]

Sorry guys this is not a issue but a question. I'm changing admin.css file and then I have to do a pull request. Do I have to change admin.css.map too? Do I have to change it manually?

[euantorano]

Hi,

The CSS is generated from the SASS files in resources/sass. Please edit these files rather than the CSS, then run the Gulp tool. For more information, please see the wiki associated with this repository.

On 28 Aug 2016, at 20:01, Nereo Berardozzi [email protected] wrote:

Sorry guys this is not a issue but a question. I'm changing admin.css file and then I have to do a pull request. Do I have to change admin.css.map too? Do I have to change it manually?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

[chack1172]

Thank you for the response euantorano :)

The currect path is resources/assets/sass

[chack1172]

@euantorano I never used sass, is it correct? Template: <table class="admin-table admin-table--bordered users"> <thead> <tr> <th colspan="2">{{ trans('admin::users.avatar_field') }}</th> </tr> </thead> <tbody> {% for user in users %} <tr> <td class="avatar-cell"> <img src="{{ user.avatar }}"> </td> <td>{{ user.name }}</td> </tr> {% endfor %} </tbody> </table>

Style `.admin-table { width: 100%;

&.admin-table--bordered {
    border-radius: 5px;
    border-collapse: separate;

    td {
        border-bottom: 1px solid $border-1;
    }
}

td, th {
    padding: 10px;

    &.avatar-cell {
        width: 44px;

        img {
            width: 44px;
            height: 44px;
        }
    }
}

thead {
    background: $primary-color;
    color: $invert-font-color;
}

}`

[euantorano]

Looks right to me, but I'll need to test it once you create a pull request :smile:

On 28 Aug 2016, at 20:48, Nereo Berardozzi [email protected] wrote:

@euantorano I nevere used sass, is it correct? Template:

{% for user in users %} {% endfor %}

{{ trans('admin::users.avatar_field') }}
	{{ user.name }}

Style `.admin-table { width: 100%;

&.admin-table--bordered { border-radius: 5px; border-collapse: separate;

td {
    border-bottom: 1px solid $border-1;
}

}

td, th { padding: 10px;

&.avatar-cell {
    width: 44px;

    img {
        width: 44px;
        height: 44px;
    }
}

}

thead { background: $primary-color; color: $invert-font-color; } }`

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

[chack1172]

You have to wait for the pull request, it is an hard edit for me :o never used laravel too

[euantorano]

Not a problem, any help is appreciated!

On 28 Aug 2016, at 20:52, Nereo Berardozzi [email protected] wrote:

You have to wait for the pull request, it is an hard edit for me :o never used laravel too

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

[chack1172]

Ok I used gulp watch to change files. There's a litte problem: - resources\assets\sass\rtl.scss <-- Not Found

[euantorano]

Interesting. I've maybe broken that one at some point. If you just commit your changed files and make a Pull Request I can fix it.

On 28 Aug 2016, at 21:39, Nereo Berardozzi [email protected] wrote:

Ok I used gulp watch to change files. There's a litte problem:

• resources\assets\sass\rtl.scss <-- Not Found

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

[chack1172]

The problem is in the file gulfile.js at line 51 https://github.com/mybb/mybb2/blob/master/gulpfile.js

The file rtl.scss doen't exists, it was changed to main.rtl.css https://github.com/mybb/mybb2/tree/master/resources/assets/sass

[euantorano]

Yeah, the file should exist, unless it's been removed recently. It provides right to left support for text in the ACP. I'll look into it, but feel free to just submit a PR with the sass.

On 28 Aug 2016, at 22:01, Nereo Berardozzi [email protected] wrote:

The problem is in the file gulfile.js at line 51 https://github.com/mybb/mybb2/blob/master/gulpfile.js

The file rtl.scss doen't exists https://github.com/mybb/mybb2/tree/master/resources/assets/sass

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

[chack1172]

How ho I set a default value to form_select if a variable doesn't exists?

{{ form_select('role', roles, role ? role.id) }}

Solved

[euantorano]

Hi, the third parameter is the default value (as seen here). I'm not sure what it is that you're trying to do, but the role ? role.id will set the default to the role's ID if the role isn't null.

[chack1172]

I meant if role was null, but now I know how to do that:

{{ form_select('role', roles, role ? role.role_slug : 'user') }}

[chack1172]

When I commit the changes, do I have to update admin.css, admin.css.map, admin_rtl.css and admin_rtl.css.map?

[euantorano]

Hi,

Gulp will do that, but it's broken at the minute. If you just commit the SASS, I can generate the CSS and map files.

[chack1172]

I'm trying to send a pull request but it select my past commit and devilshakerz commit too. How could I send only my last commit of my fork?

[euantorano]

Hi,

Did you create a new branch before starting doing your work? That's the way that we recommend doing things.

[chack1172]

No, I didn't. I didn't know that

[chack1172]

I revert changes and create a new brach

[euantorano]

Ah, we should probably make our recommendations a bit more obvious, perhaps in the README file.

On 29 Aug 2016, at 17:25, Nereo Berardozzi [email protected] wrote:

No, I didn't. I didn't know that

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

[chack1172]

Finally I've done it :) Now how do I update my master with edits?

[euantorano]

Hi,

If you push your branch, then you should be able to create a pull request from that branch to mybb2/master :smile:

[chack1172]

Ok. I've done the pull request

[chack1172]

For validation do I have to do something like the page below? https://github.com/mybb/mybb2/blob/master/app/Http/Requests/Account/UpdateProfileRequest.php

[euantorano]

Yep, something like that. For some more information, see here: https://laravel.com/docs/5.3/validation#form-request-validation https://laravel.com/docs/5.3/validation#form-request-validation

On 29 Aug 2016, at 19:08, Nereo Berardozzi [email protected] wrote:

https://github.com/mybb/mybb2/blob/master/app/Http/Requests/Account/UpdateProfileRequest.php https://github.com/mybb/mybb2/blob/master/app/Http/Requests/Account/UpdateProfileRequest.php

[chack1172]

@euantorano can you help me updating my fork with latest edits? I tried many times but it adds a new commit: https://github.com/chack1172/mybb2/commits/master

[euantorano]

Hi,

You need to do a rebase rather than a pull. For more information, please see here: https://robots.thoughtbot.com/keeping-a-github-fork-updated

We need to really document this stuff properly. I'll try and put out a dev post on our blog about our recommended workflow this week.

[chack1172]

@eyabtirabi how do I use Request validation for this? I need the id `$user = $this->userRepository->find($id);

    $validator = $this->validator->make($request->all(), [
        'name'      => 'required|max:255|unique:users,name,'.$user->id,
        'email'     => 'required|email|max:255|unique:users,email,'.$user->id,
        'password'  => 'confirmed|min:6',
        'usertitle' => 'string',
    ]);`

[euantorano]

Hi,

Is the user the current logged in user, or the user being edited? If it's the latter, you can use to IoC container to inject the user repository or model and look the user up. If you want, I can do it for you tomorrow.

[chack1172]

user being edited. Could I have an example?